Improve focus management on webview course searches

Description

Location: Course search (Native mobile apps only) e.g. https://www.edx.org/course?search_query=html

Severity: 7

Description: When adding a filter or clearing a filter element, focus is redirected to the Search field. Voiceover users have a few more swipes to get back to where they were. This may make filtering more cumbersome for VoiceOver users since they cannot select multiple filters at once. They will have to navigate back to the Filters multiple times to make their selections.

Pattern: None

Auditor Note: Developers should not shift focus to the search field. Instead, focus should go on the "Viewing results matching" heading since that also indicates what context change happened based on their interaction (added filter or removed filter).

Internal Note: Confirmed and I agree. Move focus to the Heading that reveals how many results are showing. This also affects Android.

Steps to Reproduce

None

Current Behavior

None

Expected Behavior

None

Reason for Variance

None

Release Notes

None

User Impact Summary

None

Activity

Show:
Saeed Bashir
December 4, 2017, 2:33 PM

If you are able to deploy the app to your phone locally then you can test it by updating URL for COURSE_SEARCH_URL in the configuration file.
I will test it as well and share my findings with you guys.

Waheed Ahmed
December 4, 2017, 2:40 PM
Albert (AJ) St. Aubin
December 4, 2017, 4:14 PM
Edited

There are issues with the certificate on the test site and the App I believe. The code changes were simple and releasing to production will not hurt. I am not sure how else we can test the application due to the nature of the certificate and security concern with the test site.

Can you comment as you attempted to test in app locally.

Mirza Mubashar Baig
December 5, 2017, 6:47 AM

Correct, I tried this on prod build with the configurations mentioned but on accessing 'Find Courses' screen app was showing SSL error. Seems like we cannot test this on App right at the moment.


However, this was working fine on iOS Safari.

Waheed Ahmed
December 5, 2017, 11:16 AM

Changes are now live.

Fixed

Assignee

Waheed Ahmed

Reporter

Mark Sadecki

Reach

None

Impact

None

Platform Area

None

Customer

None

Partner Manager

None

URL

None

Contributor Name

None

Groups with Read-Only Access

None

Actual Points

None

Category of Work

None

Platform Map Area (Levels 1 & 2)

None

Platform Map Area (Levels 3 & 4)

None

Story Points

2

Epic Link

Components

Sprint

None

Due date

2017/12/08

Priority

CAT-2
Configure