Skip to content

馃寭 Dark Mode - #210

Merged
tatianamac merged 27 commits into
prodfrom
feature/dark-mode
Aug 26, 2020
Merged

馃寭 Dark Mode#210
tatianamac merged 27 commits into
prodfrom
feature/dark-mode

Conversation

@ovlb

@ovlb ovlb commented Jun 12, 2020

Copy link
Copy Markdown
Collaborator

This implements dark mode. 猬涳笍

It also fixes the missing contrast outline on Windows High Contrast Mode that @dengeist reported.
And it makes the focus state inside of the coloured boxes a little more obvious.

Overall I didn鈥檛 change too much off the design because it was working quite well out of the box

@ovlb ovlb self-assigned this Jun 12, 2020
@ovlb
ovlb marked this pull request as draft June 12, 2020 09:44
@ovlb
ovlb requested a review from tatianamac June 18, 2020 20:59
@ovlb
ovlb marked this pull request as ready for review June 18, 2020 20:59
@ovlb ovlb added Status 路 Needs Review Needs editing/translation/code review and removed Status 路 In Progress labels Jun 18, 2020
@ovlb

ovlb commented Jun 18, 2020

Copy link
Copy Markdown
Collaborator Author

Note: This adds client-side scripting but no JS build process, because scope and such. Optimising JS delivery is on my radar, I will add a ticket for this asap.

@tatianamac

Copy link
Copy Markdown
Collaborator

@ovlb Bumped in the Slack channels to elicit more feedback.

@tatianamac

Copy link
Copy Markdown
Collaborator

@ovlb I think that there's one area where the text is still white on :focus, rendering it illegible. The rest looks good to me!

South is highlighted, text is white, background is pink for onfocus

@tatianamac tatianamac left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

One area in the table of contents where the word renders as white when it should be black for :focus. See comment with screenshot.

@ovlb

ovlb commented Jun 24, 2020

Copy link
Copy Markdown
Collaborator Author

@tatianamac Good catch. I鈥檝e updated the styles.

@ovlb
ovlb requested a review from tatianamac July 1, 2020 10:01
@yourpalkat

Copy link
Copy Markdown
Contributor

@ovlb Some of these highlighted terms on the documentation/front-matter page have troublesome contrast in dark mode. Everything else looks amazing to me!

Screen Shot 2020-07-04 at 7 27 03 PM

@ovlb

ovlb commented Jul 5, 2020

Copy link
Copy Markdown
Collaborator Author

@derekmurr Thanks for having a look! I鈥檝e updated the styles.

@ovlb

ovlb commented Jul 24, 2020

Copy link
Copy Markdown
Collaborator Author

@tatianamac Is there anything left to do here?

@tatianamac

Copy link
Copy Markdown
Collaborator

I love it so much!!! 馃槶

A non-blocking questions: I noticed that for the :focus mode, we have dark text on light, which causes the text colour to flip. This isn't the same behaviour on light mode:
digital Blackface is highlighted with pink highlight and red underline. Background is grey.

If this approach is intentional, then I think this is good to merge in! If it isn't, then I think making it consistent could be nice; so that the text remains the same colour.

@ovlb

ovlb commented Jul 28, 2020

Copy link
Copy Markdown
Collaborator Author

If this approach is intentional, then I think this is good to merge in!

It is, as without the colour change the contrast ratio is not sufficient.

@tatianamac

Copy link
Copy Markdown
Collaborator

If this approach is intentional, then I think this is good to merge in!

It is, as without the colour change the contrast ratio is not sufficient.

Gotcha, I guess I'm not 100% following why we can't take an inverted approach from the light mode (so, the highlight colour would need to be a darker shade.) But regardless I trust you and am fine with it as is.

@ovlb

ovlb commented Jul 28, 2020

Copy link
Copy Markdown
Collaborator Author

so, the highlight colour would need to be a darker shade

That鈥檚 a version we can try. I think I took the inverted route because I didn鈥檛 need to broaden the colour palette and I liked the effect.

@ovlb

ovlb commented Aug 22, 2020

Copy link
Copy Markdown
Collaborator Author

@tatianamac I鈥檝e updated the focus styles.

@tatianamac

Copy link
Copy Markdown
Collaborator

@ovlb Perfect! I think this is ready to ship! 馃殌

@ovlb

ovlb commented Aug 25, 2020

Copy link
Copy Markdown
Collaborator Author

@tatianamac Let鈥檚 go! 馃殺

@ovlb

ovlb commented Aug 26, 2020

Copy link
Copy Markdown
Collaborator Author

@tatianamac Do you wanna push the button?

@tatianamac
tatianamac merged commit 7f7943d into prod Aug 26, 2020
@tatianamac
tatianamac deleted the feature/dark-mode branch August 26, 2020 10:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Status 路 Needs Review Needs editing/translation/code review Type 路 Feature Request New feature or request Type 路 Frontend Frontend work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants