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

WCAG upgrade - Create test page for: Button #2171

Merged
merged 5 commits into from
Sep 7, 2023

Conversation

BrahimMahadi
Copy link
Contributor

This pull request includes a test page with working examples for the various button styles. More info can be found by referencing WET-361.

Copy link
Member

@duboisp duboisp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I completed a review

  • Fix the various request changes
  • Remove everywhere the use of <span class="com">
  • Add the equivalent example but for anchor element <a>
  • Apply the same change in French

common/button/button-en.html Outdated Show resolved Hide resolved
common/button/button-en.html Outdated Show resolved Hide resolved
common/button/button-en.html Outdated Show resolved Hide resolved
common/button/button-en.html Outdated Show resolved Hide resolved
common/button/button-en.html Show resolved Hide resolved
common/button/button-en.html Outdated Show resolved Hide resolved
common/button/button-en.html Outdated Show resolved Hide resolved
common/button/button-en.html Outdated Show resolved Hide resolved
common/button/button-en.html Outdated Show resolved Hide resolved
common/button/button-en.html Outdated Show resolved Hide resolved
@BrahimMahadi BrahimMahadi temporarily deployed to github-ci June 14, 2023 17:11 — with GitHub Actions Inactive
Copy link
Member

@duboisp duboisp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few change required in the working example and the JSON-LD need to be updated to ensure those example are added on our main index page

common/button/button-en.html Outdated Show resolved Hide resolved
common/button/button-en.html Outdated Show resolved Hide resolved
common/button/button-fr.html Outdated Show resolved Hide resolved
common/button/button-fr.html Outdated Show resolved Hide resolved
common/button/index.json-ld Outdated Show resolved Hide resolved
@BrahimMahadi BrahimMahadi temporarily deployed to github-ci July 13, 2023 18:16 — with GitHub Actions Inactive
Copy link
Member

@duboisp duboisp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Voir les change request et il faudrait appliquer les même changement de manière équivalente en français.

common/button/button-en.html Outdated Show resolved Hide resolved
common/button/button-en.html Outdated Show resolved Hide resolved
common/button/button-en.html Show resolved Hide resolved
common/button/button-en.html Outdated Show resolved Hide resolved
@BrahimMahadi BrahimMahadi temporarily deployed to github-ci August 22, 2023 18:27 — with GitHub Actions Inactive
common/button/button-en.html Outdated Show resolved Hide resolved
common/button/button-fr.html Show resolved Hide resolved
Copy link
Member

@duboisp duboisp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The French page DOM structure should match the English page DOM structure in order to have both language equivalent

For example the following need to be updated for the French page

  • Les items dans la section Sur cette page
  • Le titre "Utilisation améliorée" doit être enlever et les heading level subséquente ajusté

@BrahimMahadi BrahimMahadi temporarily deployed to github-ci September 1, 2023 18:24 — with GitHub Actions Inactive
Copy link
Member

@duboisp duboisp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I reviewed and everything are fine and do work as expected but there is still one very-little change to make. See the code suggestion.

common/button/button-fr.html Outdated Show resolved Hide resolved
@duboisp duboisp merged commit cb8dc51 into wet-boew:master Sep 7, 2023
1 check passed
SebastianBurke pushed a commit to SebastianBurke/GCWeb that referenced this pull request Mar 6, 2024
* WCAG upgrade - Create test page for: Button
* Button: Addressed Review Comments
* Updated working examples
* Addressed Requested Changes
* Matched DOM structure between both pages
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants