Include axe checks in user_views_branches_spec.rb feature test
requested to merge 426232-include-axe-checks-in-spec-features-projects-branches-user_views_branches_spec-rb-feature into master
What does this MR do and why?
- Adding accessibility checks to feature specs for Branches view
- Fixing listed errors
- A missing toggle text declared for "More action" menu trigger
- Switched ID to a class for multiple "Download" buttons
- Unifying the toggles for "More actions" menu triggers. There were present for most of the triggers, but missing in the main nav controls.
Screenshots or screen recordings
Screenshots are required for UI changes, and strongly recommended for all other merge requests.
Before | After |
---|---|
|
All errors fixed
|
How to set up and validate locally
Numbered steps to set up and validate the change are strongly suggested.
The pipeline should pass successfully.
To validate muanually:
- Download buttons that had ID, but now have a class, are preset on two views
- Go to
Code > Repository
and clickDownload
button above the file list. Verify that you can download a selected format. - Go to
Code > Branches
and clickDownload
button for any branch on the list. Verify that you can download a selected format.
- Go to
- Go to
Code > Branches
and turn on a. screen reader. Tab through the nav controls (filter search bar,View branch rules
,New Branch
,More actions
trigger) and make sure thatMore actions
is announced, when you reach the trigger. The tooltip withMore actions
should also show on hover.
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Related to #426232 (closed)
Edited by Paulina Sedlak-Jakubowska