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

feat(ui): Dark Mode #2493

Merged
merged 2 commits into from
May 25, 2024
Merged

feat(ui): Dark Mode #2493

merged 2 commits into from
May 25, 2024

Conversation

Hazer
Copy link
Member

@Hazer Hazer commented May 2, 2024

Description

Includes dark mode, auto selects and allows for manually overriding with user preference

Screenshot

Screenshot 2024-05-02 at 09 30 50 Screenshot 2024-05-02 at 09 31 48 Screenshot 2024-05-02 at 09 32 04 Screenshot 2024-05-02 at 09 32 15 Screenshot 2024-05-02 at 09 31 23 Screenshot 2024-05-02 at 09 31 33

Depends on:

Type of Change

  • New feature (non-breaking change which adds functionality)
  • Dependency update (updates to dependencies)
  • Documentation update (changes to documentation)
  • Repository update (changes to repository files, e.g. .github/...)

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated the in code docstring/documentation-blocks for new or existing methods/components

Branch Updates

LizardByte requires that branches be up-to-date before merging. This means that after any PR is merged, this branch
must be updated before it can be merged. You must also
Allow edits from maintainers.

  • I want maintainers to keep my branch updated

@Hazer Hazer changed the title Dark Mode feat(ui): Dark Mode May 2, 2024
@Hazer
Copy link
Member Author

Hazer commented May 7, 2024

I didn't ask before, but any considerations regarding colors? Specially with the NavBar colors?

@ReenigneArcher
Copy link
Member

I didn't ask before, but any considerations regarding colors? Specially with the NavBar colors?

The white(ish) on yellow(ish) is probably going to be difficult to read. I think I'd prefer the same navbar colors for both, e.g. the original. But everywhere else seems to look okay from your screenshots.

Also, I apologize for the delays on getting all these reviewed and in, I'm a bit overwhelmed with this project at the moment, having 50 open PRs. Are you on our discord? If so it would probably easier to communicate about any of these and get them across the goal line.

@Hazer
Copy link
Member Author

Hazer commented May 7, 2024

The white(ish) on yellow(ish) is probably going to be difficult to read. I think I'd prefer the same navbar colors for both, e.g. the original. But everywhere else seems to look okay from your screenshots.

Fair, will do.

Also, I apologize for the delays on getting all these reviewed and in, I'm a bit overwhelmed with this project at the moment, having 50 open PRs. Are you on our discord? If so it would probably easier to communicate about any of these and get them across the goal line.

I can totally understand, no problem. I wasn't, but I just got there, I've sent you a DM. Thanks!

@Hazer Hazer marked this pull request as draft May 7, 2024 15:01
@Hazer
Copy link
Member Author

Hazer commented May 7, 2024

Draft until #2491 is merged.

Copy link

codecov bot commented May 13, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 6.96%. Comparing base (2b18e4c) to head (4a2a791).
Report is 159 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##           master   #2493      +/-   ##
=========================================
- Coverage    7.03%   6.96%   -0.08%     
=========================================
  Files          87      87              
  Lines       17697   17691       -6     
  Branches     8406    8406              
=========================================
- Hits         1245    1232      -13     
- Misses      13716   13754      +38     
+ Partials     2736    2705      -31     
Flag Coverage Δ
Linux 5.35% <ø> (ø)
Windows 2.56% <ø> (ø)
macOS-12 ?
macOS-13 ?
macOS-14 8.27% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 22 files with indirect coverage changes

@Hazer
Copy link
Member Author

Hazer commented May 13, 2024

Screenshot 2024-05-13 at 15 40 31
Screenshot 2024-05-13 at 15 40 43

Dark mode navbar colors updated

@Hazer Hazer marked this pull request as ready for review May 13, 2024 18:42
ReenigneArcher
ReenigneArcher previously approved these changes May 14, 2024
@ReenigneArcher
Copy link
Member

I don't know why, but the toggle drop down doesn't work on the "Applications" tab.

@Hazer
Copy link
Member Author

Hazer commented May 15, 2024

I don't know why, but the toggle drop down doesn't work on the "Applications" tab.

That's odd. I have really no idea why too, there's no obvious reason for it, I'll need to debug later

@ReenigneArcher
Copy link
Member

I don't know why, but the toggle drop down doesn't work on the "Applications" tab.

That's odd. I have really no idea why too, there's no obvious reason for it, I'll need to debug later

@TheElixZammuto any idea what could cause this?

The "button" works on every other navbar page, and there are no errors in the browser console.

@Hazer
Copy link
Member Author

Hazer commented May 22, 2024

The "button" works on every other navbar page, and there are no errors in the browser console.

Fixed. It had to do with bootstrap module bundling being implicitly duplicated. It'll probably be a good idea to revamp the vite configs in the future, for now I just removed the duplication.

ref:
aurelia/skeleton-navigation#894
https://discourse.aurelia.io/t/bootstrap-import-bootstrap-breaks-dropdown-menu-in-navbar/641/9

Hazer added 2 commits May 24, 2024 21:23
chore(ui): fixes code review

chore(ui): fix dark navbar colors

chore: fixes from review

chore: rename to just theme
@ReenigneArcher ReenigneArcher merged commit 4a91301 into LizardByte:master May 25, 2024
49 checks passed
@Hazer Hazer mentioned this pull request May 30, 2024
11 tasks
KuleRucket pushed a commit to KuleRucket/Sunshine that referenced this pull request Jun 6, 2024
@Hazer Hazer deleted the feat/dark-mode branch June 7, 2024 05:22
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