-
Notifications
You must be signed in to change notification settings - Fork 238
a11y: improve accessibility on rich text editor options. #4886
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
Conversation
📱 Scan the QR code below to install the build (arm64 only) for this PR. |
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #4886 +/- ##
========================================
Coverage 80.27% 80.28%
========================================
Files 2148 2148
Lines 57045 57067 +22
Branches 7185 7189 +4
========================================
+ Hits 45793 45816 +23
Misses 8811 8811
+ Partials 2441 2440 -1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
interactionSource = remember { MutableInteractionSource() }, | ||
indication = ripple( | ||
bounded = false, | ||
radius = 20.dp, | ||
), | ||
) | ||
.size(48.dp) | ||
.then( | ||
if (toggleable) { | ||
Modifier.toggleable( |
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.
TIL 👀
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.
Thanks!
Content
Ensure that talkback (aka screen reader) give the state of the various rich text editor option. The state can be disabled, selected, or not selected, depending on the type of formatting option.
Motivation and context
Improve accessibility of the rich text editor.
Note: iOS PR is element-hq/element-x-ios#4194, but we do not need to use the strings created there, we can use the facilities provided by the compose framework instead.
Screenshots / GIFs
Tests
Tested devices
Checklist