Improve titles, move canUndo CurrentStep check into canUndoAction

src/components/DominionAssistant.tsx

@@ -32,11 +32,7 @@ const DominionAssistant: React.FC<DominionAssistantProps> = ({ route, navigation
   const { showAlert } = useAlert();
 
   useEffect(() => {
-    setCanUndo(
-      gameState.currentStep === CurrentStep.GameScreen
-        ? canUndoAction(gameState, gameState.log.length - 1)
-        : false
-    );
+    setCanUndo(canUndoAction(gameState, gameState.log.length - 1));
   }, [gameState, gameState.log]);
 
   const undoLastAction = () => {

src/components/GameScreen.tsx

@@ -41,7 +41,7 @@ const ButtonContainer = styled(Box)(({ theme }) => ({
 
 const FabContainer = styled(Box)(({ theme }) => ({
   position: 'fixed',
-  bottom: theme.spacing(2),
+  bottom: theme.spacing(10),
   right: theme.spacing(2),
   display: 'flex',
   flexDirection: 'row',

src/components/Player.tsx

@@ -1,14 +1,5 @@
 import React, { useState } from 'react';
-import {
-  Checkbox,
-  Paper,
-  Box,
-  IconButton,
-  Popover,
-  Tooltip,
-  Typography,
-  useTheme,
-} from '@mui/material';
+import { Checkbox, Paper, Box, IconButton, Popover, Tooltip, Typography } from '@mui/material';
 import { styled } from '@mui/system';
 import ArrowRightIcon from '@mui/icons-material/ArrowRight';
 import SettingsIcon from '@mui/icons-material/Settings';
@@ -23,6 +14,7 @@ import { GameLogActionWithCount } from '@/game/enumerations/game-log-action-with
 import { PlayerFieldMap } from '@/game/types';
 import { useAlert } from '@/components/AlertContext';
 import { FailedAddLogEntryError } from '@/game/errors/failed-add-log';
+import theme from '@/components/theme';
 
 const StyledPaper = styled(Paper)(({ theme }) => ({
   padding: 5,
@@ -54,7 +46,6 @@ interface PlayerProps {
 }
 
 const Player: React.FC<PlayerProps> = ({ addLogEntry }) => {
-  const theme = useTheme();
   const { gameState, setGameState } = useGameContext();
   const { showAlert } = useAlert();
   const [showNewTurnSettings, setShowNewTurnSettings] = useState(false);

src/components/SelectFirstPlayer.tsx

@@ -7,18 +7,17 @@ import {
   ListItemIcon,
   ListItemText,
   ListItemButton,
-  useTheme,
 } from '@mui/material';
 import ArrowRightIcon from '@mui/icons-material/ArrowRight';
 import { useGameContext } from '@/components/GameContext';
+import theme from '@/components/theme';
 
 interface SelectFirstPlayerProps {
   nextStep: () => void;
 }
 
 const SelectFirstPlayer: React.FC<SelectFirstPlayerProps> = ({ nextStep }) => {
   const { gameState, setGameState } = useGameContext();
-  const theme = useTheme();
 
   const selectRandomFirstPlayer = useCallback(() => {
     if (gameState.players.length > 0) {

src/components/screens/DominionAssistantScreen.tsx

@@ -1,7 +1,9 @@
 import React from 'react';
-import { Box, Typography, styled } from '@mui/material';
+import { Box, Typography } from '@mui/material';
+import { styled } from '@mui/system';
 import { useNavigate, useLocation } from 'react-router-dom';
 import DominionAssistant from '@/components/DominionAssistant';
+import SuperCapsText from '@/components/SuperCapsText';
 
 const StyledContainer = styled(Box)(({ theme }) => ({
   display: 'flex',
@@ -22,7 +24,7 @@ export default function DominionAssistantScreen() {
 
   return (
     <StyledContainer>
-      <StyledTitle variant="h4">Unofficial Dominion Assistant</StyledTitle>
+      <SuperCapsText fontSize={24}>Unofficial Dominion Assistant</SuperCapsText>
       <DominionAssistant route={location} navigation={navigate} />
     </StyledContainer>
   );

src/game/dominion-lib-undo.ts

@@ -13,6 +13,7 @@ import { NotEnoughProphecyError } from '@/game/errors/not-enough-prophecy';
 import { NotEnoughSupplyError } from './errors/not-enough-supply';
 import { NotEnoughSubfieldError } from './errors/not-enough-subfield';
 import * as undoHelpers from './dominion-lib-undo-helpers';
+import { CurrentStep } from './enumerations/current-step';
 
 /**
  * Returns the linked actions for the given log entry.
@@ -41,6 +42,10 @@ export function canUndoAction(game: IGame, logIndex: number): boolean {
     return false;
   }
 
+  if (game.currentStep !== CurrentStep.GameScreen) {
+    return false;
+  }
+
   const actionToUndo = game.log[logIndex];
 
   // Find the main action if this is a linked action