-
Notifications
You must be signed in to change notification settings - Fork 517
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
[A11y] Changes in Admin Controls Download Permissions View #2210
Comments
Hello @Arjupta , I'm working on the issue. Please assign me the issue. |
@7phalange7 assigned it to you |
@Arjupta to fix the above issues I need to add min height/weight and text color, So should I use |
You must use dimens and color files for resolving any issue. |
Fixes Issue 3 of oppia#2210, Changed the text color as per Chantel's suggesstion
Checklist(Contributors can take one issue at a time)
|
@Arjupta please assign me |
Issue -1 is ready to be worked upon. Assigned it to you. Try attaching a screenshot of before changes , after changes and test scans also in your Pull Request |
Reopening it as it got closed automatically by merging a PR which solved only a part of this Issue |
Closing as it will get solved in #2618 |
Describe the bug
Admin Control Download Permission View is not fully compatible with some users from the Accessibility point of view.
Issue 1 - The Size of Switch button must be larger ( minimum 48dpX48dp)
Issue 2 - Multiple items have the same Description as that of Switch Button
Issue 3 - The Contrast ratio of the text (foreground and background) inside the TextViews is 2.68 but the required one 4.50
To Reproduce
Steps to reproduce the behaviour:
Expected behaviour
All the tests must be passed by scanning the screenshots for both Tablet and Mobile Devices in every orientation.
Screenshots
Device
Additional context
The Screenshots are descriptive enough for understanding the issues but still, if there is a need of clarification then you can comment here. Multiple Pull Requests can also be made for solving these issues.
Reference - https://github.com/oppia/oppia-android/wiki/Working-on-UI#accessibility-scanner
The text was updated successfully, but these errors were encountered: