-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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 #11494
Comments
Thanks for opening your first issue! This space is protected by our Code of Conduct - and we're here to help. |
Hi, I'd like to try. |
I'd like to do this for the rest of the html files |
Hi, the issue has'nt been approved yet. |
Hi @Natacha-Danbadan, thank you for opening this issue. Your suggestion is very good and will help improve the accessibility of the site. Can you make a list of all the files that need this change? Thank you! |
The first set of HTML files can be found in the plots2 repo public folder, https://github.com/publiclab/plots2/tree/main/public The second set of HTML files can be found in the leaflet.distortableImage repo examples folder, https://github.com/publiclab/Leaflet.DistortableImage/tree/main/examples |
Hi @Natacha-Danbadan, this repo has a lot of HTML files but most of them don't have the normal HTML boilerplate. Can you add a list of the files with the HTML boilerplate so I can review them? Thanks! |
Yes @TildaDares I have just done that. I'm still expecting an FTO issue. Thank you ma'am. |
Hi @Natacha-Danbadan, the other HTML files in here https://github.com/publiclab/plots2/tree/main/public apart from |
Yes ma'am, I also stated the other HTML files that need to be worked on in my previous message. I'll get started on that ASAP. Thanks ma'am. |
so while working on the accessibility on the HTML files , i discovered there was no base tag on the head tag, is there a a Default URL and a default target for all links on a page you would like me to add on the HTML base tag? @TildaDares @jywarren @PeculiarE |
Hi @Natacha-Danbadan, setting a base tag now would mean having to change all the internal links in the project. That would be too big a task to complete before the contribution period ends. Thank you! |
Okay ma'am, duly noted. |
@TildaDares @cesswairimu @jywarren kindly help review my PR for the Plots2 repo. #11539 |
@TildaDares @cesswairimu @jywarren kindly help review my PR for the Leaflet.DistortableImage Repo |
Hi @Natacha-Danbadan is this issue taken id love to work on it |
This issue aims to improve the accessibility of the html pages (Semantic HTML) and fix the position of the scripts to Keep up with best practices. This to enable web crawlers and users in general to be able to access the site's content regardless of a user's impairment. For example replace the nav tag with the header tag, have a main tag in all html files, position the script tag inside of the body as some are positioned out of the body. This would be done to all html files contained in the repo below.
https://github.com/publiclab/plots2/tree/main/public
The text was updated successfully, but these errors were encountered: