-
Notifications
You must be signed in to change notification settings - Fork 98
Dark theme #160
base: master
Are you sure you want to change the base?
Dark theme #160
Conversation
Nice work, @literalpie! I look forward to checking this out in the next day or two. In the meantime, and for posterity, would you mind sharing a few screenshots here? |
I've added changes to make use of SPM resource bundling allowed in 5.3. It references an in-progress PR to update If the update to 5.3 should be it's own PR, let me know and I can split it out, or anyone can use my changes as a starting point. |
ec8e7f6
to
d21a0fd
Compare
@literalpie Really nice work! Glad this is on track. I was wondering, was omitting the color for |
d8fb3f6
to
40a8e3c
Compare
@literalpie Right, I noticed after my comment. In that case that could be changed later. @mattt Thoughts? I can do a pass on the colors after this PR is merged if you wouldn't mind. |
Matt, I've updated with support for both pre and post-5.3, the build is passing, and I have squashed my changes into one commit. I've also added logic to gracefully fall back when color-scheme queries are not supported, or when CSS custom properties are not supported. Having made these changes, as far as I know, this is ready to merge. Let me know if you have any suggestions. |
18b4d97
to
81f2d3a
Compare
Hi @mattt I'm still interested in getting this merged. Is now a good time to rebase, or are there other changes you are planning to get done before this is merged? I see that you've moved the CSS from being a bundled file to a string in a Swift file. Should I do something similar with the JS generated here? |
I welcome feedback on anything, specifically:
Hope you find this helpful!