Closed
Bug 1132768
Opened 10 years ago
Closed 10 years ago
Cursor can be placed in contenteditable=false element inside of contenteditable=true container if cursor was previously in a different contenteditable container
Categories
(Core :: DOM: Editor, defect)
Tracking
()
RESOLVED
FIXED
mozilla39
People
(Reporter: roan.kattouw, Assigned: ehsan.akhgari)
References
Details
Attachments
(1 file)
User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:35.0) Gecko/20100101 Firefox/35.0
Build ID: 20150125222008
Steps to reproduce:
See http://jsfiddle.net/8ea7aff2/
1. Click in "I am div number one" to place cursor
2. Click in the "I am not editable" in the first div; entire div is selected
3. Click in "I am div number two"
4. Click in the "I am not editable" in the first div again; cursor appears this time
Actual results:
In step 4, the cursor appeared in a ce=false area.
Expected results:
It should not be possible for the cursor to appear in a ce=false area.
Firefox prevents this correctly when the cursor is moving within ce=true container that contains the ce=false node like in #2 (it selecting the text isn't correct, but that's bug 1097242), but when the cursor moves from a different ce=true container, this is not prevented.
In Chrome, the cursor does not enter the ce=false area when you click in it, regardless of whether the selection came from the same ce=true container or a different one.
Reporter | ||
Updated•10 years ago
|
Flags: needinfo?(ehsan)
Reporter | ||
Updated•10 years ago
|
Summary: Cursor can be placed in contenteditable=false element inside of contenteditable=true container f cursor was previously in a different contenteditable container → Cursor can be placed in contenteditable=false element inside of contenteditable=true container if cursor was previously in a different contenteditable container
Assignee | ||
Updated•10 years ago
|
Assignee | ||
Updated•10 years ago
|
Component: Untriaged → Editor
Flags: needinfo?(ehsan)
Product: Firefox → Core
Assignee | ||
Comment 1•10 years ago
|
||
Assignee: nobody → ehsan
Assignee | ||
Comment 2•10 years ago
|
||
Attachment #8568404 -
Flags: review?(roc)
Attachment #8568404 -
Flags: review?(roc) → review+
Assignee | ||
Comment 3•10 years ago
|
||
Comment 4•10 years ago
|
||
sorry had to back this out for perma test failures like https://treeherder.mozilla.org/logviewer.html#?job_id=6968411&repo=mozilla-inbound
Assignee | ||
Comment 5•10 years ago
|
||
Second try: https://hg.mozilla.org/integration/mozilla-inbound/rev/99419a04c349
Had to disable selection carets in addition to touch carets.
Status: UNCONFIRMED → RESOLVED
Closed: 10 years ago
status-firefox39:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
Comment 7•9 years ago
|
||
This was backed out from Fx40/Fx41 for causing bug 1181130.
You need to log in
before you can comment on or make changes to this bug.
Description
•