Skip to content

Enhance accessibility#3

Open
Steeven9 wants to merge 3 commits into
mainfrom
enhance-accessibility
Open

Enhance accessibility#3
Steeven9 wants to merge 3 commits into
mainfrom
enhance-accessibility

Conversation

@Steeven9

@Steeven9 Steeven9 commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator
  • added preventUnload on refresh and back button to avoid navigating away by mistake
  • make texts more readable by increasing contrast
  • made checkboxes in the terrain section mirrored for enhanced layout and aestethics
  • wrote description of the PR

@Steeven9 Steeven9 requested a review from Fireblast9 June 8, 2026 11:55

@Fireblast9 Fireblast9 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Great work, however as a user I currently get prompted about confirmation to leave even when there is nothing on the page, it would make more sense to have something like a boolean or a flag that checks whether the user actually has something worth preventing losing.

Also maybe it would be nice to prompt the user also on the "back" browser button (if possible, otherwise disregard this)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants