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

Max Score on PageSpeed Insights #8

Closed
harsxv opened this issue Sep 5, 2024 · 5 comments · Fixed by #40
Closed

Max Score on PageSpeed Insights #8

harsxv opened this issue Sep 5, 2024 · 5 comments · Fixed by #40

Comments

@harsxv
Copy link
Owner

harsxv commented Sep 5, 2024

Hello!

I opened this good first issue to help first-time contributors get started.

The score on PageSpeed Insights can be improved to achieve a maximum score of 💯

Files can be checked:

  • index.html.theme
  • history.html.theme

It’s really just a matter of color contrast. I’ve adjusted it to pass the check here: https://dequeuniversity.com/rules/axe/4.9/color-contrast, but I’m not fond of the dark green against the background.

contrast analyzer darker green result
image image

Maybe someone with a stronger sense of UI/UX can help improve this. I’d appreciate your contribution!

Score currently: https://pagespeed.web.dev/analysis/https-status-harry-id/3bfccw1chu

Screenshot 2024-09-05 at 22 51 07
@Radbluelite
Copy link

Can I work on this issue to improve the accessiblity?

@harsxv
Copy link
Owner Author

harsxv commented Oct 6, 2024

Yes of course. I will assign the issue. Thank you!

@atinder-harika
Copy link
Contributor

@harsxv Since there have been no pr's for this issue I would be glad to contribute, I have good experince with UI/UX in general.

@harsxv
Copy link
Owner Author

harsxv commented Oct 28, 2024

@atinder-harika sure, I've assigned to you. Thanks!

@atinder-harika
Copy link
Contributor

@harsxv I made PR #40, resolving this issue.

@harsxv harsxv linked a pull request Oct 31, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants