-
Notifications
You must be signed in to change notification settings - Fork 132
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
Application: adding application template and examples #2267
Conversation
Pre-approved upon successful review and testing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Garneauma Il faudrait faire une petite documentation rapide afin d'officialiser la version 1 de ce template.
@GormFrank @delisma @bsouster Qu'est-ce que vous pensez de l'utilisation du double h1 pour indiquer le nom de l'application? Ou, est-ce que vous avez une solution alternative qui respecterait le modèle visuelle de cette page tout en étant conforme au WCAG 2.1. Vous pouvez voir le markup proposé via le fichier modifier de ce PR "templates/application/layouts/application.html"
Il faut considérer que ce PR a pour but principal d'éliminer l'utilisation de CSS custom dans CDTS concernant leur application template actuelle et qu'éventuellement nous allons devoir prendre le temps nécessaire afin de revoir entièrement l'application template. L'objectif ici c'est d'avoir rapidement un modèle conforme à l'accessibilité et qui maintient l'expérience visuelle qui a été implémenté dans CDTS et utilisé par les applications web utilisant CDTS.
Voici les examples pratiques
- https://universallabs.org/wet/labs/pr2267/templates/application/default-application-en.html
- https://universallabs.org/wet/labs/pr2267/templates/application/application-with-banner-en.html
Moi, @duboisp, je crois que l'utilisation du double h1 ici pourrait être satisfaisant pour ce gabarit jusqu'à ce qu'on prenne le temps de revoir ce gabarit dans son entièreté.
d075d65
to
1c9a611
Compare
@GormFrank @delisma @bsouster Après discussion avec @duboisp, l'idée d'utiliser le double H1 a été mise de côté. Le nom de l'application utilise maintenant un H2. |
1c9a611
to
81d3159
Compare
81d3159
to
99fb9c8
Compare
99fb9c8
to
fbca7f4
Compare
@Garneauma @duboisp @delisma @bsouster I agree with the H2 approach. I personnally believe that it doesn't cause any semantics problems, as well as UX. |
fbca7f4
to
83efb27
Compare
Pre-approve upon review and local testing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few item for discussion.
Also I completed a the local testing and the review. And it look good.
83efb27
to
1def262
Compare
Pre-approved upon another review and local testing. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed and tested locally,
Everything work as expected. With this change we are going to be able to initiate the removal of custom CSS used by CDTS
Adding CDTS application elements inside an "Application" template.