hugo: Modified blue tint for base styling#474
hugo: Modified blue tint for base styling#474Wies-UsMedia wants to merge 1 commit intocue-lang:masterfrom
Conversation
✅ Deploy Preview for cue ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Wies-UsMedia
requested review from
JorindeUsMedia,
JuanUsMedia and
anne-usmedia
JorindeUsMedia
left a comment
JorindeUsMedia
left a comment
There was a problem hiding this comment.
Question: is $c-blue still used anywhere after this change?
If not I would suggest assigning the value that is now assigned to $c-blue--light to $c-blue and getting rid of --light
Wies-UsMedia
force-pushed
the
feature/3503
branch
from
0730520 to
bf993ee
Compare
Updated accordingly. |
Wies-UsMedia
force-pushed
the
feature/3503
branch
from
bf993ee to
7caaf67
Compare
JorindeUsMedia
left a comment
JorindeUsMedia
left a comment
There was a problem hiding this comment.
The $c-blue--light from before which is been replaced by $c-blue is not changed in all scss files yet. I now get the very light blue as nav text for instance. Please check all usage of $c-blue--light and check if it needs to be replaced with $c-blue now.
Wies-UsMedia
force-pushed
the
feature/3503
branch
from
7caaf67 to
e96b630
Compare
I believe I caught all of them now, but let me know if you still find a few out of place. |
The darker blue tint used across the site has been replaced by the brighter blue version. To compare, see: https://cuelang.org/examples/shortcodes/info/ https://cuelang.org/examples/shortcodes/buttons/ Fixes cue-lang/cue#3503 Signed-off-by: Wies de Graaf <wies.de.graaf@usmedia.nl>
Wies-UsMedia
force-pushed
the
feature/3503
branch
from
e96b630 to
2f34701
Compare
3 participants
As per commit.
To test and compare, go to:
https://deploy-preview-474--cue.netlify.app/examples/shortcodes/info
https://deploy-preview-474--cue.netlify.app/examples/shortcodes/buttons