List outdent is not working properly

RESOLVED FIXED in mozilla9

Status

()

Core
Editor
RESOLVED FIXED
13 years ago
6 years ago

People

(Reporter: ABHIJEET BHATTACHARYA, Assigned: kaze)

Tracking

Trunk
mozilla9
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

13 years ago
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.

Comment 1

13 years ago
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/

Comment 3

12 years ago

*** This bug has been marked as a duplicate of 290026 ***
Status: UNCONFIRMED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → DUPLICATE

Comment 4

6 years ago
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 → ---
(Assignee)

Comment 5

6 years ago
Confirmed with a quick contentEditable test, this is a Core/Editor bug.

Updated

6 years ago
Assignee: composer → nobody
Component: Composer → Editor
Product: SeaMonkey → Core
QA Contact: editor
Hardware: x86 → All
Version: unspecified → Trunk

Updated

6 years ago
Status: REOPENED → NEW

Comment 6

6 years ago
Created attachment 551332 [details] [diff] [review]
Test for bug 291780

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!

Comment 8

6 years ago
Created attachment 551492 [details] [diff] [review]
mochitest-plain test for bug 291780 [Checked in: Comment 10]

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 10

6 years ago
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]

Updated

6 years ago
Flags: in-testsuite+
(Assignee)

Comment 11

6 years ago
Created attachment 556022 [details] [diff] [review]
patch proposal

http://tbpl.allizom.org/?tree=Try&usebuildbot=1&rev=ef4d7fd6a742
Assignee: nobody → kaze
Status: NEW → ASSIGNED
(Assignee)

Updated

6 years ago
Attachment #556022 - Flags: review?(ehsan)
Comment on attachment 556022 [details] [diff] [review]
patch proposal

Can this be landed?
Attachment #556022 - Flags: review?(ehsan) → review+
(Assignee)

Comment 13

6 years ago
TryServer looks ok:
https://tbpl.mozilla.org/?tree=Try&usebuildbot=1&rev=ef4d7fd6a742
http://hg.mozilla.org/integration/mozilla-inbound/rev/056dda4ab249
Target Milestone: --- → mozilla9
http://hg.mozilla.org/mozilla-central/rev/056dda4ab249
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago6 years ago
Resolution: --- → FIXED

Comment 16

6 years ago
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.