microsoft / axe-windows

Automated accessibility testing engine for Windows applications
MIT License
136 stars 62 forks source link

feat(localization): initial how to fix fixes #790

Closed codeofdusk closed 1 year ago

codeofdusk commented 1 year ago

Details

This PR adds more meaningful "how to fix" text for some rules in place of just repeating the description.

Motivation

Part of localization feature (internal access required to view).

Context

Remaining rules to follow, these are just the ones that I felt confident fixing.

Pull request checklist

codecov-commenter commented 1 year ago

Codecov Report

Merging #790 (0bca446) into main (4fc19c3) will increase coverage by 1.03%. The diff coverage is 77.46%.

@@            Coverage Diff             @@
##             main     #790      +/-   ##
==========================================
+ Coverage   73.70%   74.73%   +1.03%     
==========================================
  Files         398      421      +23     
  Lines       12046    12983     +937     
==========================================
+ Hits         8878     9703     +825     
- Misses       3168     3280     +112     
Impacted Files Coverage Δ
src/Actions/Actions/ControlPatternAction.cs 0.00% <0.00%> (ø)
src/Actions/Actions/CustomUIAAction.cs 0.00% <0.00%> (ø)
src/Actions/Actions/ListenAction.cs 0.00% <0.00%> (ø)
src/Actions/Actions/LoadActionParts.cs 90.90% <ø> (ø)
src/Actions/Actions/SetDataAction.cs 0.00% <0.00%> (ø)
...rc/Actions/Attributes/InteractionLevelAttribute.cs 0.00% <ø> (ø)
src/Actions/Contexts/ElementDataContext.cs 86.36% <ø> (ø)
src/Actions/Misc/ExtensionMethods.cs 7.46% <0.00%> (-31.60%) :arrow_down:
src/Actions/Resources/ErrorMessages.Designer.cs 0.00% <ø> (-55.56%) :arrow_down:
src/Automation/Data/OutputFile.cs 88.88% <ø> (ø)
... and 285 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codeofdusk commented 1 year ago

@ferBonnin accepted these changes offline. Merging.