iPadOS 13 Checkboxes won't select/deselect with VO

[KatieWoe]

For phetsims/qa#429
I was checking VO on iPadOS Beta since it will be coming out soon. I was unable to check and uncheck boxes (Force Values, Distance, and Constant Size). When attempting to double tap to do so, it would instead seem to activate the pink keyboard focus box and both it and the iPad's focus could be seen over the box. Repeated attempts might cause it to read the box's description again, but it would not check or uncheck the box. It seemed to work as expected in iOS 12.

[zepumph]

Assigning @jessegreenberg for assistance. Do you have a way to test this or reproduce it? I think we also may want to move this issue out of GFLB depending on its severity.

[jessegreenberg]

I confirmed it is working as expected in iOS 12. Ill see if I can get iOS 13 to test.

[jessegreenberg]

I have an iPad Air 2, https://www.apple.com/ipados/ says that will be compatible. I am not sure how getting the beta works but iPadOS comes out tomorrow, Ill take a look then.

[zepumph]

Good luck!

[terracoda]

Is the iOS on iPad different from iOS on the iPhone?
I can check and uncheck checkboxes on iPhone 7 with iOS 12.4.1
Unfortunately, I haven't upgraded yet, so not sure how useful my test is.

[KatieWoe]

It is as of today's apple update.

[KatieWoe]

Not sure what it looks like on iOS 13 on iPhone.

[jessegreenberg]

OK, I confirmed that the checkbox cannot be checked in iPadOS13 while the cursor is on the checkbox. However, if the cursor is on the checkbox label, the checkbox can be checked correctly. Ill see if this is true for native HTML

[jessegreenberg]

https://jsfiddle.net/Lbnu98xq/ works OK.

It looks like the HTML checkboxes are not sized to overlap the entire Node with label:

In chrome, the box takes up the entire text like this

Guessing width isn't being set correctly. Moving to scenery because I think this involves styling of PDOM elements.

[terracoda]

In comment phetsims/scenery#1001 (comment), @jessegreenberg has a solution, though it may be "fragile".

@zepumph, do we need to do another RC to test this out?

[zepumph]

@terracoda we will definitely want another RC for this. The commits over in phetsims/scenery#1001 will be cherry picked to the branch. Then we will have QA look at this.

[KatieWoe]

Issue seems solved. Checked by @loganbraywork