Open Bug 1208836 Opened 4 years ago Updated Last year

Invalid element passed to the method IsElementClickableAndReadable

Categories

(Firefox for Android :: General, defect)

42 Branch
defect
Not set

Tracking

()

People

(Reporter: domivinc, Assigned: domivinc)

Details

Attachments

(1 file)

Testing the fix of the bug 1208370, I also found another issue in the method IsElementClickableAndReadable. In [1], there are some large buttons with a text and an icon (for instance the button “API >”). When the click occurs on the icon, the method returns that the element is unreadable. In this case, the zoomed view is wrongly displayed.
It’s because the tested frame in this method is not the correct one. It must be the frame linked to the clickable element (the container element of the text + the icon) and not the frame linked to the touch point element (the icon).
This issue is more visible in this page because the jquery-mobile library uses an invisible span element (with the entity  ) to display the icon. 

[1] http://demos.jquerymobile.com/1.3.2/widgets/buttons/
Assignee: nobody → domivinc
I fixed this error passing the clickable content element in the calls to IsElementClickableAndReadable.
Attachment #8666457 - Flags: review?(bugmail.mozilla)
Comment on attachment 8666457 [details] [diff] [review]
patch-27092015 Ter 2-Bug_1208836___Invalid_element_passed_to_the_method_IsElementClickableAndReadable__r_cats.patch

Review of attachment 8666457 [details] [diff] [review]:
-----------------------------------------------------------------

Same, please add a test and re-request review. Also s/cats/kats/
Attachment #8666457 - Flags: review?(bugmail.mozilla)
You need to log in before you can comment on or make changes to this bug.