Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Generate and store dark CSS file #346

Merged
merged 2 commits into from
Oct 27, 2023
Merged

Generate and store dark CSS file #346

merged 2 commits into from
Oct 27, 2023

Conversation

trasher
Copy link
Member

@trasher trasher commented Oct 23, 2023

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Oct 23, 2023

Codecov Report

Merging #346 (ebb60f6) into develop (0878ed0) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@              Coverage Diff              @@
##             develop     #346      +/-   ##
=============================================
- Coverage      44.18%   44.18%   -0.01%     
- Complexity      5902     5903       +1     
=============================================
  Files            142      142              
  Lines          23915    23917       +2     
=============================================
  Hits           10567    10567              
- Misses         13348    13350       +2     
Files Coverage Δ
galette/lib/Galette/Core/Authentication.php 86.81% <0.00%> (-1.96%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@trasher
Copy link
Member Author

trasher commented Oct 24, 2023

This PR is far to be finished, but I already cause some issues:

  • some image paths are incorrect
  • dark theme would be enabled for everyone.
  • it's currently enabled per default and should be on demand only.

@trasher
Copy link
Member Author

trasher commented Oct 24, 2023

Also, maybe using the extension for users who need it is also a good solution (and requires 0 changes Galette side).

Add Dark Reader in js dependencies
Generate and store dark CSS file
Better name for included file

Rely on browser configuration to switch theme

Tested with:
- firefox
- chrome (using chrome://flags/#enable-force-dark)
@trasher trasher marked this pull request as ready for review October 26, 2023 20:58
@trasher trasher merged commit ebb60f6 into develop Oct 27, 2023
37 checks passed
@trasher trasher deleted the feature/darkmode branch October 27, 2023 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants