Skip to content

Conversation

naveenkumar29052006
Copy link

Added a color mode toggle button in the sidebar, placed to the left of the Logout button.

Supports light and dark themes based on system preference.

Persist user’s preference in localStorage.

Handles adding/removing dark/light CSS classes dynamically.

Provides a Context and Provider for managing theme state across the app.
Screenshot 2025-10-08 at 7 06 48 PM

Copy link

netlify bot commented Oct 8, 2025

Deploy Preview for hoppdocs ready!

Name Link
🔨 Latest commit c7c1350
🔍 Latest deploy log https://app.netlify.com/projects/hoppdocs/deploys/68e8155db4141e000899f27f
😎 Deploy Preview https://deploy-preview-133--hoppdocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@CLAassistant
Copy link

CLAassistant commented Oct 8, 2025

CLA assistant check
All committers have signed the CLA.

@naveenkumar29052006
Copy link
Author

@konsalex
check it

Copy link
Contributor

@konsalex konsalex left a comment

Choose a reason for hiding this comment

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

Hey @naveenkumar29052006 thanks for the PR. Before reviewing locally could you please do some cleanup as there are unrelated changes to the root package and port configuration.

Also sign the CLA please. Thanks

- POSTGRES_DB=hopp
ports:
- '5432:5432'
- '5433:5432'
Copy link
Contributor

Choose a reason for hiding this comment

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

This PR should not change dev setup

SESSION_SECRET=renkey-hash-key
REDIS_URI=redis://localhost:6379/0?protocol=3
DATABASE_DSN=postgres://hopp:password@localhost:5432/hopp?sslmode=disable
DATABASE_DSN=postgres://hopp:password@localhost:5433/hopp?sslmode=disable
Copy link
Contributor

Choose a reason for hiding this comment

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

The same as above

@naveenkumar29052006
Copy link
Author

@konsalex all required changes done sir and by this toggle button i created this issue #134

Copy link
Contributor

@konsalex konsalex left a comment

Choose a reason for hiding this comment

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

Will review shortly @naveenkumar29052006 .

Builds are failing with:

the six-letter p-prefixed code.
9:11:43 PM: ➤ YN0086: │ Some peer dependencies are incorrectly met by dependencies; run yarn explain peer-requirements for details.
9:11:43 PM: ➤ YN0028: │ The lockfile would have been modified by this install, which is explicitly forbidden.
9:11:43 PM: ➤ YN0000: └ Completed
9:11:43 PM: ➤ YN0000: · Failed with errors in 0s 251ms
9:11:43 PM: Error during Yarn install
9:11:43 PM: Failing build: Failed to install dependencies

I see this is because root yarn.lock was modified with irrelevant packages, can you also clean it?

@naveenkumar29052006
Copy link
Author

sir could you please check now

Copy link
Contributor

@konsalex konsalex left a comment

Choose a reason for hiding this comment

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

After the cleanup update the root yarn.lock

@naveenkumar29052006
Copy link
Author

You were right sir i mistakenly changed dependencies of docs and tauri and i had ran yarn also so could u please check now

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.

3 participants