If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Tapping SVG shape invokes text selection UI

REOPENED
Unassigned

Status

()

Core
Selection
REOPENED
a year ago
11 months ago

People

(Reporter: Glen Reesor, Unassigned)

Tracking

(Blocks: 1 bug)

48 Branch
ARM
Android
Points:
---

Firefox Tracking Flags

(firefox47 unaffected, firefox48 wontfix, firefox49 affected, fennec+, firefox50 affected)

Details

Attachments

(1 attachment)

(Reporter)

Description

a year ago
Created attachment 8770998 [details]
svgIssue.html

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:47.0) Gecko/20100101 Firefox/47.0
Build ID: 20160604131506
Firefox for Android

Steps to reproduce:

Using Firefox 48 beta on Android Lollipop, open the attached html file. Tap the circle or rectangle.


Actual results:

The UI associated with selecting text appears--"markers" at each end of the shape appear as if there's text in between, and there's a bar across the top with the check mark for copying text.


Expected results:

Nothing should have happened, as the user is not selecting text. Note: This does not happen in Android Firefox 47.0.
(Reporter)

Updated

a year ago
OS: Unspecified → Android
Hardware: Unspecified → ARM
(Reporter)

Updated

a year ago
Component: Untriaged → General
Product: Firefox → Firefox for Android

Comment 1

a year ago
I'm able to reproduce this using:

Device:Galaxy Note 3(5.0), Galaxy Note 5(6.0),Xiaomi Pad 2(5.02)
Build:Beta - 48.0b6, Aurora - 49.0a2(2016-07-13), Nightly - 50.0a1(2016-07-14)
Status: UNCONFIRMED → NEW
status-firefox48: --- → affected
status-firefox49: --- → affected
status-firefox50: --- → affected
Ever confirmed: true
Mark this looks to be fallout from the new text selection handles. 47 is unaffected.
tracking-fennec: --- → ?
status-firefox47: --- → unaffected
status-firefox48: affected → wontfix
tracking-firefox49: --- → ?
tracking-firefox50: --- → ?
Flags: needinfo?(markcapella)
My quick thought was dup of [0] ?

With the original UI scheme, it was our working assumption (at least as recently as [1]) that non-text bearing selections were ignoreable.

The rules did indeed change with AccessibileCarets, and we've let this bit of incompatiblility remain gray-ish.

Perhaps TYLin would care to comment further? I'm sure code can accommodate front-end one way or another.


[0] Bug 1221335 - AccessibleCarets seem to select some things (images, etc) that contain no text
[1] Bug 1119834 - Text selection action bar is shown on div element containing no text
Flags: needinfo?(markcapella)
Component: General → Text Selection
Flags: needinfo?(tlin)
Duping this makes sense.
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → DUPLICATE
Duplicate of bug: 1221335
Unsetting tracking nom for 49/50. We will track in the duplicate bug.
tracking-firefox49: ? → ---
tracking-firefox50: ? → ---
Blocks: 1124074
Component: Text Selection → Selection
Flags: needinfo?(tlin)
Product: Firefox for Android → Core
I think this is a separate issue from Bug 1221335. I can reproduce this issue on desktop browser.

When single-clicking on the SVG circle or rectangle, AccessibleCaretManager::OnSelectionChanged() is called with a non-collapsed selection range. That's why the carets are shown.

The question is: why is *single-clicking* on the SVG shape resulting a selection?

[1] http://searchfox.org/mozilla-central/rev/ff0e3782b1c44ffd1a68ef8e9e4f8be3866741e2/layout/base/AccessibleCaretManager.cpp#137
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
tracking-fennec: ? → +
Duplicate of this bug: 1297112

Comment 8

11 months ago
Just to add, I am still seeing this issue on Firefox 49.0.2 on Android 6.0.1 (Samsung Galaxy S7).
You need to log in before you can comment on or make changes to this bug.