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

Update documentation color scheme #209

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

Conversation

kafitzgerald
Copy link
Collaborator

@kafitzgerald kafitzgerald commented Feb 8, 2024

PR Summary

Updates the documentation color scheme for more contrast using colors from the NSF NCAR brand standards.

Hopefully we can decide on something here that could then apply to all GeoCAT docs.

Alternately, we could take a look at this first on GeoCAT Applications or GeoCAT Examples since those have some additional visual elements.

@kafitzgerald kafitzgerald added the blocked Work got blocked waiting the output of some other source/work label Feb 21, 2024
@kafitzgerald kafitzgerald removed the blocked Work got blocked waiting the output of some other source/work label Oct 16, 2024
@kafitzgerald kafitzgerald marked this pull request as ready for review October 16, 2024 21:35
@kafitzgerald kafitzgerald marked this pull request as draft October 16, 2024 21:57
@kafitzgerald
Copy link
Collaborator Author

Once we the link checker and pre-commit passing again I'm going get this updated.

I'm hoping we can decide on something here and apply it to the other GeoCAT branded projects or vice versa.

I was reminded of this by some color scheme issues on geocat-applications that would be nice to get fixed (e.g. with the highlighting on tables that's difficult to read).

@kafitzgerald kafitzgerald added the documentation Improvements or additions to documentation label Oct 17, 2024
@kafitzgerald kafitzgerald marked this pull request as ready for review October 17, 2024 16:51
@kafitzgerald kafitzgerald marked this pull request as draft October 21, 2024 16:39
@jukent
Copy link
Collaborator

jukent commented Nov 1, 2024

Hi @kafitzgerald just checking in, since you asked for a review on this but it's marked as draft.

@kafitzgerald
Copy link
Collaborator Author

Ah, sorry for the confusion. I should have added a note as well when I marked this as draft.

I ended up doing something slightly different after discussing w/ Anissa and looking at GeoCAT-applications. I was planning to port that over to other packages at some point, but wanted to give it a bit of time to see how we liked it first.

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

Successfully merging this pull request may close these issues.

2 participants