cynkra / cynkrablog

Source of the cynkra blog
https://cynkra.com/blog
0 stars 1 forks source link

split dark/light modes #45

Closed maelle closed 9 months ago

maelle commented 10 months ago

Fix #36 Fix #6

TODOS

maelle commented 9 months ago

I'm a bit stuck here.

@christophsax who could help with

christophsax commented 9 months ago

@DivadNojnarg, perhaps you could help here?

maelle commented 9 months ago

@DivadNojnarg at the moment for the light theme we use https://cynkra.com/assets/img/logo_small.svg

Docs of dark theme customization https://quarto.org/docs/output-formats/html-themes.html#dark-mode

Question on Quarto forum https://github.com/quarto-dev/quarto-cli/discussions/8379

maelle commented 9 months ago

Regarding the dark mode toggle, currently invisible in light mode. Using the web dev console I see that it responds to "dark" theme things whereas I am looking at it in light mode?!

image

github-actions[bot] commented 9 months ago

Summary

Status Count
🔍 Total 937
✅ Successful 888
⏳ Timeouts 0
🔀 Redirected 0
👻 Excluded 48
❓ Unknown 0
🚫 Errors 1

Errors per input

Errors in docs/index.html

maelle commented 9 months ago

@DivadNojnarg for info https://github.com/quarto-dev/quarto-cli/discussions/8601

DivadNojnarg commented 9 months ago

@maelle. I am currently checking the branch.

$border-dark-color: rgba($black, .4) !default;
$card-border-color: $border-dark-color !default;

I believe we should have: $border-dark-color: rgba($white, .4) !default;.

This can be set with $navbar-fg: rgba($white, .8); if you don't need full white (see below)

Screenshot 2024-02-07 at 15 31 29
DivadNojnarg commented 9 months ago

@maelle @christophsax. Is this what you expect?

https://github.com/cynkra/cynkrablog/assets/18291543/cb05a026-1fcf-4178-b81d-0a5fee0a07ae

DivadNojnarg commented 9 months ago

What still remain is the look of the navbar dropdown elements to make them in line with cynkra.com (rounded border, hover background color, ...)

Screenshot 2024-02-08 at 11 40 08
maelle commented 9 months ago

@DivadNojnarg regarding the card border, I might have been badly imitating https://cynkra.com/consulting/ where there are no borders, but it looks better with borders indeed.

maelle commented 9 months ago

The logo switch is beautiful :star_struck:

maelle commented 9 months ago

waiting for an answer on https://github.com/quarto-dev/quarto-cli/discussions/8601 before I tweak further

maelle commented 9 months ago

Thanks SO MUCH @DivadNojnarg!!