Conversation
|
Color definitions for Finto: |
osma
left a comment
There was a problem hiding this comment.
This looks very good! A lot of visual improvements in this PR, great work!
The finto.css rules need to be wrapped within :root { ... } for them to actually work. It took a while for me to figure out this...
Regarding the triangle icons: I wonder if another implementation would be possible here. The current bitmap PNG images look a bit fuzzy to me - maybe SVG would be crisper? I also looked at suitable Font Awesome icons - there are a few triangles and carets (e.g. caret-right) available in the Free section that we use, but no outline versions, so maybe that's not an option. Anyway, the Finto layout needs different colors for the triangle icons, so if we go with image files, another set of images is needed for Finto.
|



Reasons for creating this PR
CSS variables in Skosmos 3 are not currently set to support multiple color schemes. There are also some small issues with the layout and CSS. This PR creates new color definitions and fixes some remaining issues with CSS.
Link to relevant issue(s), if any
Description of the changes in this PR
Known problems or uncertainties in this PR
Checklist
.sr-onlyclass, color contrast)