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 color theme pokemon-go #87

Closed
wants to merge 2 commits into from
Closed

Add color theme pokemon-go #87

wants to merge 2 commits into from

Conversation

remichoi
Copy link

Response to Issue #12

Screenshot of my A11y card:

For assistance: https://help.github.com/en/articles/file-attachments-on-issues-and-pull-requests

@TerabyteTiger
Copy link
Owner

Hi!

It looks like you added the theme to the CSS file, but not to ThemePicker.vue

If you need assistance, you can find a guide on the full process here: https://dev.to/tvanblargan/make-your-first-pull-request-3iai

Drop a comment once that's updated and I'll remove the invalid tag + merge 😄

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

Sorry for that! I've updated it now and git pushed, please let me know if there are any more issues. Thank you for your patience!

@TerabyteTiger
Copy link
Owner

I'm seeing the theme now, however it's currently not WCAG compliant.

If you could update the theme so that the 3 sentences in the "A11y" card show as AA or AAA complaint, I can merge your theme in!

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