Closed Bug 291780 Opened 15 years ago Closed 8 years ago

List outdent is not working properly

Categories

(Core :: Editor, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla9

People

(Reporter: abhijeet.bhattacharya, Assigned: kaze)

Details

Attachments

(2 files, 1 obsolete file)

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: 14 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
Status: REOPENED → NEW
Attached patch Test for bug 291780 (obsolete) — Splinter Review
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]
Flags: in-testsuite+
Attached patch patch proposalSplinter Review
http://tbpl.allizom.org/?tree=Try&usebuildbot=1&rev=ef4d7fd6a742
Assignee: nobody → kaze
Status: NEW → ASSIGNED
Attachment #556022 - Flags: review?(ehsan)
Comment on attachment 556022 [details] [diff] [review]
patch proposal

Can this be landed?
Attachment #556022 - Flags: review?(ehsan) → review+
http://hg.mozilla.org/mozilla-central/rev/056dda4ab249
Status: ASSIGNED → RESOLVED
Closed: 14 years ago8 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.