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
fix(a11y): tag link & header logo link #111
Conversation
@yokinist is attempting to deploy a commit to the Craigary Team on Vercel. A member of the Team first needs to authorize it. |
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/craigary/nobelium/Drf7eJ4Sv8mP8CVtt1cCzRvFAmYs |
Hi, thank you for the PR 😘 I will take a look tomorrow~ meanwhile, could you update layouts/full width.js as well? They're basically the same, but for wider layout. |
LGTM, Merged! |
Thanks! |
fix(a11y): tag link & header logo link
Thank you for develop awesome blog system 💯
I sent PR for the following reasons:
-> fixed link to use next/router for accessibility
The style of selected tag was not applied on search page.
(New) Header logo was not wrapped with
a
tag