Skip to content

Add light theme support#2

Open
jln-brtn wants to merge 4 commits intoadonisjs:mainfrom
jln-brtn:main
Open

Add light theme support#2
jln-brtn wants to merge 4 commits intoadonisjs:mainfrom
jln-brtn:main

Conversation

@jln-brtn
Copy link

@jln-brtn jln-brtn commented Feb 28, 2026

🔗 Linked issue

No issue opened

❓ Type of change

  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • 👌 Enhancement (improving an existing functionality like performance)
  • ✨ New feature (a non-breaking change that adds functionality)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

This PR introduces a white theme switchable via a sun/moon toggle button in the header. The selected theme is persisted in localStorage and applied immediately on page load (anti-FOUC inline script) to avoid any flash.

Out of scope / follow-up:
The front/home page has not yet been fully styled for light mode — some assets are dark mode only.

📝 Checklist

  • I have linked an issue or discussion.
  • I have updated the documentation accordingly.

Julien Breton added 4 commits February 28, 2026 14:27
- Updated README to include build process instructions.
- Enhanced CSS with dark mode variants and new theme styles.
- Implemented theme toggle functionality in JavaScript, persisting user preference in localStorage.
- Modified layout and component templates to support dynamic theme changes.
- Improved accessibility and visual consistency across components in light and dark modes.
@RomainLanz
Copy link
Member

Hi @jln-brtn! 👋🏻

It looks good!
I believe the only missing asset is the round dot. It should be easy to make a white version of it. Do you want to give it a try ? 😄

@jln-brtn
Copy link
Author

jln-brtn commented Mar 6, 2026

Hey @RomainLanz ,
Thanks! Sure, I can manage a white dot 😉 . What about the hero banner ? Can stay in black if you want.

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