Open
Bug 1248186
Opened 9 years ago
Updated 1 year ago
Caret disappears in [contenteditable] element if I press Right arrow at the end of text
Categories
(Core :: DOM: Editor, defect)
Core
DOM: Editor
Tracking
()
REOPENED
119 Branch
| Tracking | Status | |
|---|---|---|
| firefox47 | --- | wontfix |
| firefox48 | --- | wontfix |
| firefox49 | --- | wontfix |
| firefox50 | --- | wontfix |
| firefox51 | --- | wontfix |
| firefox-esr68 | --- | wontfix |
| firefox-esr102 | --- | wontfix |
| firefox-esr115 | --- | wontfix |
| firefox75 | --- | wontfix |
| firefox76 | --- | wontfix |
| firefox77 | --- | wontfix |
| firefox117 | --- | wontfix |
| firefox118 | --- | wontfix |
| firefox119 | --- | affected |
People
(Reporter: arni2033, Assigned: pommicket)
References
()
Details
(Keywords: regression)
Attachments
(2 files)
>>> My Info: Win7_64, Nightly 47, 32bit, ID 20160210071115
STR:
1. Open the following "data:" url or click URL in the form above
> data:text/html,<div class="declaration"><span></span><span class="value" contenteditable="">linear-gradient(to left, red, blue, green, yellow), url(test.png)</span> <span></span></div>
2. Click on the text on that page (preferably - between "s" and "t" in word "test")
3.A) Hold Right arrow for several seconds
3.B) Press Right arrow 6 times (if you followed advice in Step 2)
AR: Caret disappears. Looks like it just leaves [contenteditable] element
ER: Caret should be displayed at the end of text in [contenteditable] element
This is regression from bug 669026. I'm not sure, so leaving "regressionwindow-wanted" keyword.
Regression range:
> http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=c5b90ea7e475&tochange=f63a99195987
|
||
Comment 1•9 years ago
|
||
Your suspected bug looks highly likely to be the culprit, yes.
Keywords: regressionwindow-wanted
|
||
Updated•9 years ago
|
Flags: needinfo?(bugs)
|
|
||
Updated•9 years ago
|
status-firefox48:
--- → affected
|
|
||
Comment 2•9 years ago
|
||
jet, we'd like to get this fixed but we're getting it off the 47 tracking lists.
status-firefox49:
--- → affected
status-firefox50:
--- → affected
Fairly minor regression since 2011. It would be nice to fix but we are not going to track it.
|
||
Updated•5 years ago
|
|
||
Updated•5 years ago
|
status-firefox75:
--- → affected
status-firefox76:
--- → affected
|
|
||
Updated•5 years ago
|
|
||
Updated•4 years ago
|
notes (and workaround):
- the
contenteditableis not affected when it has a border - also when when it is the last child, hence the
.in this proof of concept:
data:text/html,<div><span contenteditable="" style="border: 1px solid black">edit me!</span>.</div>
|
||
Updated•2 years ago
|
Severity: normal → S3
Adjust nsFocusManager::GetSelectionLocation so that
it handles the case of moving to the end of a contenteditable
(previously only handled form controls)
|
||
Updated•1 year ago
|
Assignee: nobody → pommicket
Status: NEW → ASSIGNED
|
||
Comment 7•1 year ago
|
||
This patch shouldn't change behavior. It just removes a lot of noise
from that function.
In particular, it simplifies various conditions that can't happen, like
startNode being a text-node but a form control at the same time, or a
text-node and the root element.
Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6fd2f0219394
Preliminary clean-up of nsFocusManager::GetSelectionLocation. r=masayuki
|
||
Comment 9•1 year ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
status-firefox119:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 119 Branch
Just landed the preparation patch.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
|
||
Updated•1 year ago
|
|
||
Updated•1 year ago
|
status-firefox117:
--- → wontfix
status-firefox118:
--- → wontfix
status-firefox-esr102:
--- → wontfix
status-firefox-esr115:
--- → wontfix
You need to log in
before you can comment on or make changes to this bug.
Description
•