Closed Bug 402284 Opened 12 years ago Closed 12 years ago
Switching between design
Mode and content Editable broken
When turning on designMode on a page that has contentEditable elements the elements that don't have a contentEditable attribute can't be edited. This is a regression caused by the patch for bug 389350.
I've tested Peter's patch, and it does fix the case he pointed out. It also fixes the case that Karl pointed out here: https://bugzilla.mozilla.org/show_bug.cgi?id=389350#c21 Maybe we should push this into 1.9?
You mean M9? It should certainly block 1.9.
Yes, I mean M9; seeing as we were trying to get the fix for 389350 into M9, we may as well put this patch in that fixes it properly.
Target Milestone: --- → mozilla1.9 M9
Comment on attachment 287177 [details] [diff] [review] Untested patch I've tested this patch, and it WFM, can you review this so we can push it into M9 please Brade?
Attachment #287177 - Flags: review?(brade) → superreview+
Since I wrote the patch I don't think I should review, but Chris should be able to imho.
Comment on attachment 287177 [details] [diff] [review] Untested patch Looks good to me.
Attachment #287177 - Flags: review?(brade) → review+
Comment on attachment 287177 [details] [diff] [review] Untested patch a=beltzner for M9
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
verified fixed using Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9a9pre) Gecko/2007110604 Minefield/3.0a9pre and on Tiger as well. I verified using the testcase attached to the bug.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.