-
Notifications
You must be signed in to change notification settings - Fork 62
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added dark mode to the core spec #2672
base: main
Are you sure you want to change the base?
Conversation
Actually, I did some more elaborate colour settings in the common.css file. Also checked them for the WAI colour contrast requirements. |
Would it make sense to start this off on the notes and then migrate to the rec docs once we're good with it? (Or maybe those don't have the same scale of custom styling?) |
I would prefer to change all our publications in a big bang but, pragmatically, doing it first on notes is doable, while doing it on the rec track documents becomes more complicated because it requires a more complicated publication process. So, yours may be the right pragmatic approach... I actually think that this should be a group decision and not ours as editors. (@wareid @shiestyle something to discuss at a call?)
I actually did not check that; I do not know whether they all include a reference to |
I'm not sure, either. I think, unfortunately, some may inline the custom css even though they don't need it. That's why I'm not sure we could even test all the styling even if the notes purport to use it. At some point we should try and clean out the unnecessary use of custom styling, and maybe look at ways of optimizing it for the docs that need it. I suspect it's often called on only to get the zebra table styling. |
Yep. And we should get all the notes to use the same Let us try to get an agreement with the WG about the dark theme (or not) and we can take care of that cleanup at the same time. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Working now on my end!
The code sample colours are set by the "official" W3C style sheet. I would not change that (just to be consistent with other documents), although, if we want, we can do it. The official style sheet put the image background to white. I was a bit bothered by this glaring white spot, so I set it to slightly yellowish, but we can change that. The reason of the white background for the images is that the figures are usually prepared for a white background, and they do not necessarily look good on black background. In the VC WG I spent quite some time on creating images that work on both backgrounds, and it is extremely difficult, so I can see why the choice was to keep it white. |
* main: updated snapshot add info about corrections remove duplicate para fix comment period end fix respec errors updated snapshot add pc para to sotd remove custom sotd para duplicated in export fix pubrules errors add proposed correction release typo Change candidate corrections to proposed rename candidate corrections release folder consolidate release folders
I played with this for the core spec. The necessary changes are:
We can play with a more elaborate change in colours for those entries (caution, explanation, etc), but I think @mattgarrish has more experience in making properly contrasted colours. I did the minimal that worked for me.
I do not think we should merge this PR now, just make it ready to merge. First, we should have all documents prepared to dark mode. Also, we may want to wait until we begin working on the EPUB 3.4 line...
See:
Preview | Diff