-
Notifications
You must be signed in to change notification settings - Fork 527
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] Revision Card #2637
Comments
i want to work on this issue |
@Prince-kushwaha I have assigned you two issues right now. So not assigning you this issue for now. Let's wait for your other PRs to get merged. |
@rt4914 I want to work on this issue. |
@omooooori Thanks for showing interest. Please checkout our wiki first. This will help you in setting up the project and once you sign the CLA and fill the survey form we can assign you issues to work on. |
@rt4914 may I work on this? |
@rt4914 I signed CLA and answered survey. So if it is ok, could you please assign this task to me. |
@rt4914 I tested this app on physical device (Samsung C7 pro API 26) as well as on emulator(Pixel 2 API 29) But Only Read Oppia once when we open the what is fraction under Revision Card . Here is One More Issue when this Sound icon is tapped talkback says Unlabelled |
@omooooori Assigned you one part of the issue. |
@priyanka0906 Assigned you one part of the issue. |
@Akshat-gour Thanks for checking. What you are trying out is an exploration and this issue is related to |
@Akshat-gour It should not read Oppia at all instead it should read only the title of the screen. |
@rt4914 Sorry. I have simple question. |
@omooooori the instructions varies for different devices. You can search about enabling TalkBack in a device which you use. There are some shortcuts combination of volume and power button also( again specific for different devices) |
@omooooori I have just updated the assigned issue which makes it much simple. You just need a static label instead of a dynamic label. |
@rt4914 in my emulator ,talkback is not navigating to all the items, as shown in the video. Android.Emulator.-.Pixel_4_5554.2021-02-18.00-01-46.mp4 |
@priyanka0906 |
okay...thankyou |
@rt4914 I have tried android:accessibilityTraversalAfter="@id/revision_card_toolbar" but it didn't work....would you please provide any hint or something which would help me. |
@priyanka0906 Suggest checking out |
okay thankyou |
@rt4914 in this also title is not selected after back arrow Android.Emulator.-.Pixel_4_5554.2021-02-20.10-55-48.mp4 |
@priyanka0906 Thanks for providing the video. I think I will have to try to solve this and then only I can confirm the exact reason for the issue. |
okay sir |
@rt4914 If the first part of the issue is still available, then please assign it to me. Also, please tell the static |
@Sparsh1212 First part needs confirmation on another issue from Chantel. Also, I suggest not picking these issues as you have already finished multiple issues where you are adding label. Instead try to pick something new. |
Sure, I have commented on other different issues as well, will start working on them as assigned. |
@rt4914 I tried to resolve according to your commnet. I think Or you mean we have to not to use LiveData to set sub topic title ? Thank you in advance. |
@omooooori I have made this issue simpler. You can introduce a static string which reads |
@rt4914 I would like to take this issue if it's not yet complete can I get a little more context on what I'll have to do? |
@rt4914 I'm trying to solve part 1 bug. I used the app with talkback. So when I click on something to raise it says oppia n then the name of the unit eg: What is a fraction?. I have to replace oppia with skill page right don't touch the header of the revision card n also write test for the implementation. Please correct if I've misunderstood the issue. |
@rt4914 sir I wrote the code that says skill page instead of oppia. Do I have to write a test for the same or is the change is enough? |
…ty (#3011) * Added accessibilty label to the RevisionCardActivity * Made improvements to the RevisionCardActivityTest file * Resolved ktlint issue * Made the requested changes * Made the requested changes * Changed from subTopicId to subtopicId * Improved accessibility * Added todo back Co-authored-by: Jonathan Alvares <[email protected]>
Current Output
current_revision_card.mp4
Issues Identified
Note: To understand the above issue(s) completely, it is recommended that you setup Talkback and play with the app keeping it on and that will give you better context.
Accessibility Guide: https://github.com/oppia/oppia-android/wiki/Accessibility-(A11y)-Guide
The text was updated successfully, but these errors were encountered: