Skip to content
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

Add light mode colors #11

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

ckaiser
Copy link

@ckaiser ckaiser commented May 9, 2024

Hi! I was editing the documentation CSS for my own personal use (light text on a dark background nukes my eyes something fierce) so I figured I'd contribute it back to the project to help any other weirdos like myself.

It's using the default CSS color-scheme preference which should pick up the system scheme automatically, although I could also add a button toggle somewhere if necessary.

Here's a comparison:

color-scheme-before-after.mp4

@RisingThumb
Copy link
Contributor

RisingThumb commented May 9, 2024

Could you also try different link colours for the light mode? Maybe the same blue colour as the Godot icon? The contrast for links is pretty bad for the navbar imo. Apart from that looks visually good to me.

Can you also include examples of what <pre>, <table> and <code> blocks end up looking like? You can see these on class reference and in code snippets throughout the docs

A button toggle would also be nice if it's an easy addition

@RhapsodyInGeek
Copy link
Contributor

I could also add a button toggle somewhere if necessary.

Yes, this would be necessary. The "dark" mode should be the default as well, as the color scheme is also part of the plugin's "branding" (all of the colors used are Quake palette colors).

I imagine since it's just CSS it should work offline locally with no problems as well?

@ckaiser
Copy link
Author

ckaiser commented May 10, 2024

Alright, I'll switch it around to just use a class and a button. Question: is there any particular reason as to why the docs are using iframes? They're kinda janky for this since they all have their own DOM and it makes cascading CSS and stuff like navigating to parts of the documentation in new tabs a bit cumbersome. Have you considered using a site generator like Hugo to generate the HTML? Would make it easier to write too since you can just use markdown instead of raw html.

@RisingThumb
Copy link
Contributor

See #5 and #6 about this. The short of it is that it shouldn't be a single mega html file, it should still run offline and locally, and ideally it shouldn't require running any more dependencies or extra build steps.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants