Page MenuHomePhabricator

Localize and make accessible new strings & controls
Closed, ResolvedPublic

Assigned To
Authored By
Tsevener
Mar 15 2019, 7:17 PM
Referenced Files
F28513418: IMG_0129.PNG
Mar 29 2019, 11:44 PM
F28513416: IMG_0128.PNG
Mar 29 2019, 11:44 PM
F28513417: IMG_0131.PNG
Mar 29 2019, 11:44 PM
F28513392: IMG_0133.PNG
Mar 29 2019, 11:44 PM
F28513413: IMG_0127.PNG
Mar 29 2019, 11:44 PM
F28513419: IMG_0132.PNG
Mar 29 2019, 11:44 PM

Description

New strings for this feature still need to be localized and accessibility labels added to the controls.

The new find & replace navigation bar might need to be reworked to be accessible via Voice Over.

Test the new find & replace bar with dynamic text scaling as well. Right now it may not handle gracefully if that font scales up.

Needs to be checked on RTL languages too.

Event Timeline

ABorbaWMF subscribed.

Testing on 6.2.1 (1594)

Everything looked pretty good with dynamic text sizing. VoiceOver worked for the most part but I found that portions of the background article are 'tappable' when looking at the editing onboarding screens. For the below example, the box is the VoiceOver area and it, in this case, it speaks a portion of the background article.

IMG_0133.PNG (2×1 px, 286 KB)

I also tried some editing Hebrew for RTL, there are a few strings yet to be translated

IMG_0127.PNG (2×1 px, 282 KB)

IMG_0128.PNG (2×1 px, 368 KB)

IMG_0131.PNG (2×1 px, 388 KB)

IMG_0129.PNG (2×1 px, 433 KB)

IMG_0132.PNG (2×1 px, 384 KB)

Moving to blocked or waiting for now. I imagine we can break this into smaller tickets as well. Let me know :)

I'm unable to reproduce the tappable background on the edit onboarding screens. I tested on iPad/iPhone develop branches and 1594 TestFlight builds. Unassigning myself in case someone else is able to repro.

Could no longer reproduce the issue with Voice Over on 6.2.1 (1594)

I imagine the strings translation is coming soon :)

JMinor claimed this task.