Skip to content

fix: stuck hover on focus for header and restart button (@byseif21)#7434

Open
byseif21 wants to merge 5 commits intomonkeytypegame:masterfrom
byseif21:fix/stuck-hover
Open

fix: stuck hover on focus for header and restart button (@byseif21)#7434
byseif21 wants to merge 5 commits intomonkeytypegame:masterfrom
byseif21:fix/stuck-hover

Conversation

@byseif21
Copy link
Contributor

starting the test while hovering header buttons or restart button leave them stuck in a hover on focus

@monkeytypegeorge monkeytypegeorge added the frontend User interface or web stuff label Jan 23, 2026
@Miodec Miodec added the waiting for update Pull requests or issues that require changes/comments before continuing label Feb 27, 2026
Copilot AI review requested due to automatic review settings February 27, 2026 20:03
@github-actions github-actions bot added waiting for review Pull requests that require a review before continuing and removed waiting for update Pull requests or issues that require changes/comments before continuing labels Feb 27, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes UI elements getting visually “stuck” in hover styling when the test enters focus mode (cursor hidden / focus state applied), specifically for header controls and the restart/config UI on the test page.

Changes:

  • Consolidates focus-mode opacity rules in test.scss and includes #restartTestButton in the elements hidden during main.focus.
  • Disables pointer interactions on header.focus to prevent hover state from persisting when entering focus mode.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
frontend/src/styles/test.scss Hides additional test-page controls (including restart) in focus mode via opacity.
frontend/src/styles/nav.scss Prevents hover/click interactions on the header while in focus mode using pointer-events: none.

Copy link
Member

@Miodec Miodec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When quick restart is disabled, tabbing over to the restart test button doesn't make it visible.

@Miodec Miodec added waiting for update Pull requests or issues that require changes/comments before continuing and removed waiting for review Pull requests that require a review before continuing labels Feb 28, 2026
@github-actions github-actions bot removed the waiting for update Pull requests or issues that require changes/comments before continuing label Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend User interface or web stuff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants