Open Bug 1248187 Opened 8 years ago Updated 3 years ago

Bogus node in [contenteditable] element is deleted on page load, and added on Ctrl+Z

Categories

(Core :: DOM: Editor, defect, P5)

1.9.1 Branch
defect

Tracking

()

Tracking Status
firefox47 --- wontfix
firefox48 --- wontfix
firefox49 --- fix-optional
firefox50 --- fix-optional

People

(Reporter: arni2033, Unassigned)

References

Details

(Keywords: regression)

User Story

There were 2 regressions of contentEditable element on page [3] (STR in comment 0)

1) bug 395340 (between 2007-10-25 and 2007-10-26, bug list [1])
 Browser started to add linebreak in the beginning even if there was no \n in HTML

2) bug 611182 (between 2010-11-14 and 2010-11-15, regression range [2])
 Browser started_to_hide that linebreak, making this bug not so noticeable


> [1] https://bugzilla.mozilla.org/buglist.cgi?list_id=13136399&resolution=---&resolution=FIXED&resolution=INVALID&resolution=WONTFIX&resolution=DUPLICATE&resolution=WORKSFORME&resolution=INCOMPLETE&resolution=SUPPORT&resolution=EXPIRED&resolution=MOVED&chfieldto=2007-10-26&query_format=advanced&chfield=resolution&chfieldfrom=2007-10-25&chfieldvalue=FIXED&bug_status=UNCONFIRMED&bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&bug_status=RESOLVED&bug_status=VERIFIED&bug_status=CLOSED
> [2] http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=674f2ed15cea&tochange=edf41ff32f08
> [3] data:text/html,<!DOCTYPE html><html><body contentEditable=true><div>asdf</div></body></html>
>>>   My Info:   Win7_64, Nightly 47, 32bit, ID 20160210071115
STR:
1. Open attachment 424374 [details]
2. Press Ctrl+Z

AR: "\n" is added before the text in [contenteditable] element

ER: Either A or B
 A) [contenteditable] element shuld act like textarea, i.e. display "\n"s after page load.
    But there should be no added text on Ctrl+Z because I haven't even edit the text (sic)
 B) Ok, let [contenteditable] element delete "\n"s but Ctrl+Z shouldn't add them from nowhere

This is regression from bug 611182. I'm not sure, so leaving "regressionwindow-wanted" keyword.
Regression range:
> http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=674f2ed15cea&tochange=edf41ff32f08
Your suspected bug looks highly likely to be the culprit, yes.
FWIW I can't see us backing out bug 611182 at this point.
Flags: needinfo?(bugs)
Depends on: 1277113
Flags: needinfo?(bugs)
Sorry, I used ambiguous link for testing. I should've used url [3] (in "user story") from the start.
I described 2 regressions of testcase in "user story".
Blocks: 395340
User Story: (updated)
Version: Trunk → 1.9.1 Branch

Bulk-downgrade of unassigned, untouched DOM/Storage bug's priority.

If you have reason to believe, this is wrong, please write a comment and ni :jstutte.

Severity: normal → S4
Priority: -- → P5
You need to log in before you can comment on or make changes to this bug.