Skip to content

Improve settings readability, fix theme sync bugs, and enhance timer interactions#147

Merged
RoderickQiu merged 13 commits into
RoderickQiu:masterfrom
Fideow2:master
May 15, 2026
Merged

Improve settings readability, fix theme sync bugs, and enhance timer interactions#147
RoderickQiu merged 13 commits into
RoderickQiu:masterfrom
Fideow2:master

Conversation

@Fideow2
Copy link
Copy Markdown
Contributor

@Fideow2 Fideow2 commented May 15, 2026

Hi @RoderickQiu,

Thank you for maintaining wnr🤓🤓🤓 — I've been using it daily and really appreciate its clean design and focus-friendly philosophy!😍😍😍

While using it, I noticed a few UI/UX friction points and theme synchronization issues that I thought I could help improve.🤔🤔🤔 This PR bundles several small but meaningful fixes and enhancements that should make the app feel more polished without changing its core character.🥹🥹🥹

Changes

Bug Fixes

  • Fix theme color sync for main interface text: Custom theme colors previously caused the main interface text to revert to default colors unexpectedly. Now text colors properly persist with the selected theme.
  • Fix theme color sync for statistics page: The "Usage Statistics" page and its charts did not reflect custom theme colors. Colors now synchronize correctly across all UI components.
  • Fix button hit area alignment: Main interface button touch targets were misaligned with their visible text labels, making them difficult to interact with. Hit areas now properly match the visual elements.
  • Fix mini mode access after timer completion: In simple timer mode, the mini mode entry remained clickable after the timer finished, causing unexpected behavior. The mini mode entry is now properly disabled when the timer completes.

UI/UX Improvements

  • Collapse verbose setting descriptions: Lengthy feature descriptions in settings were visually overwhelming. Added a collapsible "?" button that expands to show detailed explanations on demand.
  • Adjust settings typography hierarchy: Text sizes in settings were inconsistently scaled and difficult to scan. Restructured font sizing to follow clear visual hierarchy.
  • Relocate back button to top-left: The back button was positioned counter-intuitively. Moved it to the top-left corner for more natural navigation flow.
  • Center pause button in mini mode: The pause button in mini mode was off-center. Centered it for better visual balance.
  • Rename "Skip" to "Reset" in stopwatch mode: The "Skip" label in stopwatch mode was semantically misleading since the action resets the timer rather than skipping a phase. Renamed to "Reset" for clarity.

Testing

  • Verified theme color persistence across app restarts
  • Confirmed statistics page reflects custom colors correctly
  • Tested collapsible descriptions on various screen sizes
  • Validated button hit areas on touch and mouse interactions
  • Confirmed mini mode is inaccessible after timer completion in simple mode

Each change is in its own commit, so feel free to cherry-pick or comment on individual ones if anything needs adjustment.😘😘😘 I'm happy to iterate based on your feedback!🫡🫡🫡

Best regards ❤️❤️❤️

Fideow2 and others added 13 commits May 15, 2026 17:51
… lower left corner to enhance the user experience
…e mini mode after skipping

I found that in the "Simple Timing" mode, if you click "Skip", the page will switch to "Completed", but the icon in the upper right corner does not return, which causes the mini mode to still be accessible and potentially cause system bugs.
Restore the intended timer, settings, and package metadata behavior after unwanted follow-up edits.
@RoderickQiu RoderickQiu self-requested a review May 15, 2026 17:59
Copy link
Copy Markdown
Owner

@RoderickQiu RoderickQiu left a comment

Choose a reason for hiding this comment

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

Hi, thanks for your contributions. Some changes are unwanted, so I reverted in one single commit (e.g. those that are inappropriate when you are on macOS, since this is a cross-platform app; and some edge cases caused by universal style changes). Other changes are warmly welcomed.

@RoderickQiu RoderickQiu merged commit 90049b0 into RoderickQiu:master May 15, 2026
3 checks passed
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