-
Notifications
You must be signed in to change notification settings - Fork 77
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
Accessibility: Add skip to content button #586
base: develop
Are you sure you want to change the base?
Conversation
I don't know what to think of this, I have never seen a button like this in my life. 🤷♂️ |
If you press "tab" on websites, this link will be displayed. Etc. https://css-tricks.com/how-to-create-a-skip-to-content-link/ Of course, my PR is useless as long as these bugs exist. |
@yannicka Cool!!! I learned something new. :-) Well, I think the bug with the |
Great :) Yes, the issue seems very similar if not identical. |
Where does the "content" start in this case? On catalog pages (categories, search, etc.) I guess the content would be the products? What if a category don't have any products but sub categories with products in them? Where is the "content" on the start page? This will become very theme and module dependent. Even with hummingbird and standard PS modules on the start page (as an example) they can freely change positions and/or activate/deactivate modules. I'm not opposed to the idea as such, but I think it's a quirky one to implement correctly since the very nature of PrestaShop is customizability the "content" in this case can (and will) different for different shops. |
The link depends on the theme, as it leads to the tag with the id The content may indeed be different between pages, but it's often considered to be what comes after the header. Which is the case here, the tag is just after the header: hummingbird/templates/layouts/layout-both-columns.tpl Lines 25 to 36 in 733a70c
On some sites, the skip link is more complex and may contain several links to take you to different places, but this would require more work. As it stands, considering that the content is always in the |
Don't push it as it is: firstly, I don't know if the naming is right or if I'm placing the elements in the right place (in terms of both HTML and CSS). Secondly, on some pages, like the category page for example, clicking on the link redirects to
/[object History]
. I haven't yet been able to find out why, so if anyone has any ideas, I'd be grateful for any help.Screenshot: