Skip to content

Feat/lotanna dark mode - #21

Open
lota001 wants to merge 2 commits into
raphgm:mainfrom
lota001:feat/lotanna-DarkMode
Open

Feat/lotanna dark mode#21
lota001 wants to merge 2 commits into
raphgm:mainfrom
lota001:feat/lotanna-DarkMode

Conversation

@lota001

@lota001 lota001 commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

What changed
Added a new DarkMode.tsx feature that provides a reusable dark/light theme toggle for PinPointPro.
Implemented a hook to manage theme state, persist user preference, and apply the correct theme class to the document.
Added a button component that can be integrated into the existing UI for an easy on/off switch.

Why it improves the project
Enhances usability by giving users a native dark mode option, which is a common and expected feature for modern productivity apps.
Improves accessibility and comfort for low-light workflows, making the app more pleasant for extended use.
Adds a meaningful functional extension without disrupting the existing architecture and keeps the feature scope isolated to a dedicated component and hook.

How it was tested
Verified that theme preference persists across refreshes by toggling dark/light mode and reloading the page.
Confirmed the appropriate class names are applied to the document root.
Checked that the toggle text and icon update correctly when switching themes.

This contribution is a feature development change that extends PinPointPro’s functionality in a well-scoped way

raphgm
raphgm previously approved these changes Jul 12, 2026
@lota001
lota001 dismissed raphgm’s stale review July 12, 2026 20:18

The merge-base changed after approval.

raphgm
raphgm previously approved these changes Jul 12, 2026
@lota001
lota001 dismissed raphgm’s stale review July 12, 2026 20:58

The merge-base changed after approval.

raphgm
raphgm previously approved these changes Jul 13, 2026

@raphgm raphgm 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.

good work

@lota001
lota001 dismissed raphgm’s stale review July 13, 2026 13:15

The merge-base changed after approval.

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