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 paper theme #94

Closed
wants to merge 1 commit into from
Closed

add paper theme #94

wants to merge 1 commit into from

Conversation

ckouder
Copy link

@ckouder ckouder commented Oct 29, 2019

@ckouder
Copy link
Author

ckouder commented Oct 29, 2019

#12

@TerabyteTiger
Copy link
Owner

Hi!

This is a great start, but currently the header doesn't have enough contrast between the text and background color.

If you could update that and ping me in a comment here once updated, I can approve the PR and merge it in 😄

@TerabyteTiger TerabyteTiger added the invalid This doesn't seem right label Oct 30, 2019
@ckouder
Copy link
Author

ckouder commented Nov 1, 2019

Do you mean the text color is too similar to the link color? @12vanblart

@TerabyteTiger
Copy link
Owner

The contrast between the Header text color and the Header background color is too low, so users may have a hard time being able to read the words.

You can find more about contrast here: https://webaim.org/resources/contrastchecker/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants