Closed Bug 291780 Opened 16 years ago Closed 10 years ago
List outdent is not working properly
User-Agent: Mozilla/5.0 (OS/2; U; Warp 4.5; en-US; rv:1.7.5) Gecko/20041220 Build Identifier: Mozilla/5.0 (OS/2; U; Warp 4.5; en-US; rv:1.7.5) Gecko/20041220 List outdent is not working properly Reproducible: Always Steps to Reproduce: 1.Open composer create a list. 2.indent the list twice or thrice. 3.Now indent part of the list twice. 4.Outdent this part. Actual Results: Outdenting the part creates three different list of originally one list. Expected Results: Outdenting should fit the part into original list and the list count should be one.
Reproducible with Mozilla 1.8b1 and Mozilla 1.8b2/20050424.
This is an automated message, with ID "auto-resolve01". This bug has had no comments for a long time. Statistically, we have found that bug reports that have not been confirmed by a second user after three months are highly unlikely to be the source of a fix to the code. While your input is very important to us, our resources are limited and so we are asking for your help in focussing our efforts. If you can still reproduce this problem in the latest version of the product (see below for how to obtain a copy) or, for feature requests, if it's not present in the latest version and you still believe we should implement it, please visit the URL of this bug (given at the top of this mail) and add a comment to that effect, giving more reproduction information if you have it. If it is not a problem any longer, you need take no action. If this bug is not changed in any way in the next two weeks, it will be automatically resolved. Thank you for your help in this matter. The latest beta releases can be obtained from: Firefox: http://www.mozilla.org/projects/firefox/ Thunderbird: http://www.mozilla.org/products/thunderbird/releases/1.5beta1.html Seamonkey: http://www.mozilla.org/projects/seamonkey/
*** This bug has been marked as a duplicate of 290026 ***
Status: UNCONFIRMED → RESOLVED
Closed: 16 years ago
Resolution: --- → DUPLICATE
This is not dealt with by bug 290026, that is for the whole list. What happens in this one is that when indenting part of a list <ul> in inserted for each one, when outdenting, instead of removing the <ul>, the list is split into 3 parts. So originally it is: <ul style="margin-left: 40px;"> <li>Item One</li> <li>Item Two</li> <li>Item Three</li> <li>Item Four<br> </li> </ul> After the indentation of part of the list: <ul style="margin-left: 40px;"> <li>Item One</li> <ul> <li>Item Two</li> <li>Item Three</li> </ul> <li>Item Four<br> </li> </ul> After the outdentation of that part of the list: <ul style="margin-left: 40px;"> <li>Item One</li> </ul> <ul> <ul> <li>Item Two</li> <li>Item Three</li> </ul> </ul> <ul style="margin-left: 40px;"> <li>Item Four<br> </li> </ul>
Status: RESOLVED → REOPENED
Ever confirmed: true
Resolution: DUPLICATE → ---
Confirmed with a quick contentEditable test, this is a Core/Editor bug.
Assignee: composer → nobody
Component: Composer → Editor
Product: SeaMonkey → Core
QA Contact: editor
Hardware: x86 → All
Version: unspecified → Trunk
This test: * Takes a 4 item bullet list. * Selects items 2 and 3. * Indents the selected items. * Tests the HTML is correct. * Outdents the selected items. * Tests the HTML is back to original. Once this bug is fixed both tests should pass but at the moment the second one fails.
Ian, thanks for writing the test case. Is there any chance that you can modify it to be a simple HTML mochitest though? There is nothing specific to <xul:editor> in this bug, so I would rather the test case to be a simple mochitest HTML file with a contenteditable area... Thanks!
This time only with HTML and no xul:editor
Attachment #551332 - Attachment is obsolete: true
Comment on attachment 551492 [details] [diff] [review] mochitest-plain test for bug 291780 [Checked in: Comment 10] If you can change the failing check to todo_is, we can land the test right now. :-)
Attachment #551492 - Flags: review+
Comment on attachment 551492 [details] [diff] [review] mochitest-plain test for bug 291780 [Checked in: Comment 10] Checked in with last test changed to "todo_is" http://hg.mozilla.org/mozilla-central/rev/42992432ed56 Once bug is fixed, need to change this to "is"
Attachment #551492 - Attachment description: mochitest-plain test for bug 291780 → mochitest-plain test for bug 291780 [Checked in: Comment 10]
Assignee: nobody → kaze
Status: NEW → ASSIGNED
Comment on attachment 556022 [details] [diff] [review] patch proposal Can this be landed?
Attachment #556022 - Flags: review?(ehsan) → review+
TryServer looks ok: https://tbpl.mozilla.org/?tree=Try&usebuildbot=1&rev=ef4d7fd6a742
Target Milestone: --- → mozilla9
Status: ASSIGNED → RESOLVED
Closed: 16 years ago → 10 years ago
Resolution: --- → FIXED
While it is certainly nice to see some of these old bugs fixed, it doesn't help much for using css in mail composition. No way AFAICT to set useCSS. @@ -4110,17 +4110,17 @@ nsHTMLEditRules::WillOutdent(nsISelectio else if (useCSS) Just a thought; Isn't it about time include more CSS editing capability for mailnews, or access to the "Full editor" Maybe that would alleviate some of the horrible (msword generated) code I'm seeing in some mail, and the bugs that go with that.
You need to log in before you can comment on or make changes to this bug.