The default bug view has changed. See this FAQ.

Image in contentEditable div auto selects it's self when it's in text without spaces around it

RESOLVED FIXED in Firefox 16

Status

()

Core
Selection
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Sam, Assigned: mats)

Tracking

({regression})

13 Branch
mozilla17
x86_64
All
regression
Points:
---
Bug Flags:
in-testsuite ?

Firefox Tracking Flags

(firefox13 wontfix, firefox14 wontfix, firefox15- wontfix, firefox16+ fixed)

Details

Attachments

(2 attachments)

(Reporter)

Description

5 years ago
Created attachment 641586 [details]
ImageBug.html

User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:13.0) Gecko/20100101 Firefox/13.0.1
Build ID: 20120615112143

Steps to reproduce:

1. Click on the editable div
2. The image automatically selects itself

If I add a space on either side of the image it's fine but without any spaces it selects itself again.


Actual results:

Image automatically selects itself


Expected results:

Nothing
(Reporter)

Updated

5 years ago
OS: Linux → All

Comment 1

5 years ago
Regression window(m-c)
Good:
http://hg.mozilla.org/mozilla-central/rev/c7101dec8deb
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:11.0a1) Gecko/20111220 Firefox/11.0a1 ID:20111220083550
Bad:
http://hg.mozilla.org/mozilla-central/rev/a8506ab2c654
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:11.0a1) Gecko/20111220 Firefox/11.0a1 ID:20111220085450
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=c7101dec8deb&tochange=a8506ab2c654

Regression window(m-i)
Good:
http://hg.mozilla.org/integration/mozilla-inbound/rev/feaccb6a4c35
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:11.0a1) Gecko/20111219 Firefox/11.0a1 ID:20111219235256
Bad:
http://hg.mozilla.org/integration/mozilla-inbound/rev/0aa9c3a5b7e1
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:11.0a1) Gecko/20111219 Firefox/11.0a1 ID:20111220011653
Pushlog:
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=feaccb6a4c35&tochange=0aa9c3a5b7e1

Triggered by: Bug 619273
Blocks: 619273
Status: UNCONFIRMED → NEW
Component: Untriaged → Selection
Ever confirmed: true
Keywords: regression
Product: Firefox → Core

Updated

5 years ago
Attachment #641586 - Attachment mime type: text/plain → text/html
(Assignee)

Comment 2

5 years ago
Created attachment 641653 [details] [diff] [review]
fix

The image is part of the spellchecking selection range.
I think the fix is to not create an selection overlay at all
for images unless they are in the primary selection.

https://tbpl.mozilla.org/?usebuildbot=1&tree=Try&rev=4c7d247904d2
Assignee: nobody → matspal
Attachment #641653 - Flags: review+
(Assignee)

Comment 3

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/bc8981c2ca73
Flags: in-testsuite?
Target Milestone: --- → mozilla17
https://hg.mozilla.org/mozilla-central/rev/bc8981c2ca73
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
status-firefox13: --- → affected
status-firefox14: --- → affected
status-firefox15: --- → affected
status-firefox16: --- → affected
tracking-firefox15: --- → ?
tracking-firefox16: --- → ?
Comment on attachment 641653 [details] [diff] [review]
fix

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

A low-risk patch to fix a Web-facing regression. Seems like a good fit for Aurora. The regression doesn't seem serious enough to warrant landing in beta.
Attachment #641653 - Flags: approval-mozilla-aurora?

Updated

5 years ago
tracking-firefox15: ? → -
tracking-firefox16: ? → +

Comment 6

5 years ago
Comment on attachment 641653 [details] [diff] [review]
fix

[Triage Comment]
Agreed - this isn't a significant enough regression to take on beta. Approving for Aurora 16, however.
Attachment #641653 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
(Assignee)

Comment 7

5 years ago
https://hg.mozilla.org/releases/mozilla-aurora/rev/a31767ccfc1b
status-firefox13: affected → wontfix
status-firefox14: affected → wontfix
status-firefox15: affected → wontfix
status-firefox16: affected → fixed
You need to log in before you can comment on or make changes to this bug.