Closed Bug 28491 Opened 25 years ago Closed 24 years ago

unlisting adds another bullet instead of getting out of list mode

Categories

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

Product:
Core
Component:
DOM: Editor
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: sujay, Assigned: sfraser_bugs)

References

Details

(Whiteboard: fix in hand) 

using 2/18 build of mozilla

1) launch mozilla
2) launch editor
3) enter some text
4) make it a bullet list
5) add more items
6) on the last item, click the list button to get out of lists

notice it doesn't get that last list item out of list, instead
it adds another bullet list(next level bullet) next to it..

I was expecting it to throw me out of list and back to normal text.

this is a regression

all platforms...
It may be a bug (it's actually working as I meant it to, but we may want to 
change it), but it's not a regression.  There aren't any previous mozillas that 
did it differently.  You have to go back to 4.x to see it done differently.



*** This bug has been marked as a duplicate of 13513 ***
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → DUPLICATE
I don't feel this is a DUP of 13513....
REOPEN..
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
reassign to jfrancis
Assignee: brade → jfrancis
Status: REOPENED → NEW
Keywords: beta1
m14
Target Milestone: M14
accepting bug
Status: NEW → ASSIGNED
Whiteboard: [PDT-]
*** Bug 13919 has been marked as a duplicate of this bug. ***
ok, i think i'm full of crap.  I do think this used to work before.  Anyway, the 
real problem here is that we are relying on the state of the toolbar list button.  
It calls RemoveList() if it is "on", and MakeList() if it isn't.  The state of 
the button, however, is often wrong.  So instead of RemoveList() getting called, 
MakeList() gets called.

This is a great bug to farm out to someone.  It probably doesn't require much of 
a brain dump from me to get started on this one.

adding EB to status summary...ender blocker...
Whiteboard: [PDT-] → [PDT-] EB
Kathy, should I m15 this one?

remove beta1 and PDT-; move to M15; reassign to simon
Assignee: jfrancis → sfraser
Status: ASSIGNED → NEW
Keywords: beta1
Whiteboard: [PDT-] EB
Target Milestone: M14 → M15
Toolbar buttons should now faithfully reflect the list state.So does this still 
happen? Perhaps it should be dupped to bug 13513?

nope. this is still a problem exactly as the original steps show it.

I can easily reproduce this in 3/16 builds...
I have a fix for this.
Status: NEW → ASSIGNED
Whiteboard: fix in hand
Fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago → 24 years ago
Resolution: --- → FIXED
verified in 4/7 build.
Status: RESOLVED → VERIFIED
*** Bug 13513 has been marked as a duplicate of this bug. ***
You need to log in before you can comment on or make changes to this bug.