Skip to content

Comments

chore: Update board item's i18n strings#337

Merged
johannes-weber merged 1 commit intomainfrom
jowejowe/update-board-items-i18n-strings
Mar 3, 2025
Merged

chore: Update board item's i18n strings#337
johannes-weber merged 1 commit intomainfrom
jowejowe/update-board-items-i18n-strings

Conversation

@johannes-weber
Copy link
Member

@johannes-weber johannes-weber commented Feb 27, 2025

Description

Add instructions to temp disable keyboard navigation while resizing/moving.
Why? A screen reader's keyboard navigation mode may intercept arrow key presses. We can't disable that with an appropriate role, that's why we extend the drag handlers description with that information.

Updated copy got approved, see CR-180476213.

Related links, issue #, if available:

  • CR-180476213
  • AWSUI-60203

How has this been tested?

  • verified updated description on VoiceOver
Review checklist

The following items are to be evaluated by the author(s) and the reviewer(s).

Correctness

  • Changes include appropriate documentation updates.
  • Changes are backward-compatible if not indicated, see CONTRIBUTING.md.
  • Changes do not include unsupported browser features, see CONTRIBUTING.md.
  • Changes were manually tested for accessibility, see accessibility guidelines.

Security

Testing

  • Changes are covered with new/existing unit tests?
  • Changes are covered with new/existing integration tests?

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Add instructions to temp disable keyboard navigation while resizing/moving.
Why? A screen reader's keyboard navigation mode may intercept arrow key presses. We can't disable that with an appropriate role, that's why we extend the drag handlers description with that information.
@codecov
Copy link

codecov bot commented Feb 27, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.71%. Comparing base (a5c49fe) to head (d8981c9).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff            @@
##             main     #337    +/-   ##
========================================
  Coverage   95.71%   95.71%            
========================================
  Files          61       61            
  Lines        4321     4321            
  Branches      669      456   -213     
========================================
  Hits         4136     4136            
  Misses        185      185            

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@johannes-weber johannes-weber marked this pull request as ready for review February 28, 2025 08:36
@johannes-weber johannes-weber requested a review from a team as a code owner February 28, 2025 08:36
@johannes-weber johannes-weber requested review from avinashbot and connorlanigan and removed request for a team and connorlanigan February 28, 2025 08:36
@johannes-weber johannes-weber added this pull request to the merge queue Mar 3, 2025
Merged via the queue into main with commit 2fbee71 Mar 3, 2025
41 of 42 checks passed
@johannes-weber johannes-weber deleted the jowejowe/update-board-items-i18n-strings branch March 3, 2025 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants