Added dark mode to the website#4439
Conversation
…ixed areas where the teme didnt apply properly.
markmandel
left a comment
There was a problem hiding this comment.
Ooh, exciting. Looking forward to seeing the preview.
/gcbrun
| @@ -0,0 +1,27 @@ | |||
| /* Dark mode overrides for the custom Agones navbar and UI */ | |||
There was a problem hiding this comment.
Apache licence please 👍🏻
| @@ -0,0 +1 @@ | |||
| @import "darkmode-overrides"; No newline at end of file | |||
|
/gcbrun |
|
Build Failed 😭 Build Id: 3493ae89-c28b-4476-aeb8-79dbe6cd305c Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
|
/gcbrun |
|
Build Succeeded 🥳 Build Id: a9a9fdbf-8c33-4357-9438-e5e557d329b1 The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version: |
|
This is nice! Just looking for a second pair of eyes -- @lacroixthomas do we care that the dark mode for the landing page is just the header?
I'm thinking we don't actually care - flipping the entire home landing page seems like a lot of work, and not really worth it. What do you think @thegalacticlor ? Did you look at the _index.html, or also figured it was too much work 😄 |
|
I didn't want to change the set colors for both that and the community page since they already feature a combination of both color palettes already, so I just focused mostly on making sure it was consistent :D |
|
Sounds reasonable to me! We will need the Apache licences at the top of the files please before we can accept the PR. But otherwise this looks good to me. |
|
That's pretty nice !! I'll definitively use it hehe Other than the Apache Licenses, looks good to me 👌🏼 |
|
@thegalacticlor if you get a moment to update the Apache licence, that would be awesome. I want to merge #4438 after this, but want to make sure any conflicts fall on my shoulders rather than yours 👍🏻 |
…thegalacticlor/agones into Adding-Dark-Mode-to-the-Website
|
Just added the Apache licenses! |
|
/gcbrun |
|
Build Succeeded 🥳 Build Id: 6fd2aa00-83ce-43b0-9c6e-461f635363bd The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version: |
Co-authored-by: Mark Mandel <mark@compoundtheory.com>
Co-authored-by: Mark Mandel <mark@compoundtheory.com>
|
Fixed! |
markmandel
left a comment
There was a problem hiding this comment.
Let's get dark.
/gcbrun
|
/gcbrun |
|
Build Failed 😭 Build Id: 4704c5a8-7a2a-452c-82b8-9676ddae0327 Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
|
flake: /gcbrun |
|
Build Succeeded 🥳 Build Id: 32697486-c62a-4c56-95ea-6f0e12e4849f The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version: |
* Added dark mode to the website- added the toggle for the mode, then fixed areas where the teme didnt apply properly. * Added apache licence * Update site/assets/scss/_darkmode-overrides.scss Co-authored-by: Mark Mandel <mark@compoundtheory.com> * Update site/assets/scss/_styles_project.scss Co-authored-by: Mark Mandel <mark@compoundtheory.com> --------- Co-authored-by: Mark Mandel <mark@compoundtheory.com> Co-authored-by: Sivasankaran R <sivasankaranr@google.com>

What type of PR is this?
/kind feature
What this PR does / Why we need it:
This adds a dark mode to the website by adding a toggle in the nav bar based on Docsy's. This also makes theme consistent across all pages, fixing certain colors and overwritten variables.
Which issue(s) this PR fixes:
Closes #4429
Special notes for your reviewer:
All files created were made so I could properly override the variables already set.