1.open composer 2.insert list, make sure you have at least 3 list items, add text to each list item 3.highlight all of the text in the list, press any key what happens: one of the additional bullet items remains, instead of just the first item expected: only the first list item should remain
setting to m19
Target Milestone: --- → M19
accepting this bug as if it were my own son. which it is.
Status: NEW → ASSIGNED
sometimes this happens, sometimes it doesnt. i believe what's happening is this: if the last list item has a trailing br, you wont see it (the br ends a line but does not cause a new line: you need more text after it or another br for that to happen). When you select sometimes you end up selecting that last be and sometimes you don't. When you dont, it ends up in a list item all by itself, like it should. The real problem is that there is no way for the user to see the br, or know if it's selected. cc'ing mike. Mike, i think if you drag select past a text node, going below and to the right of it, you sometimes don't get the br following it selected. Maybe we should always select it in that case.
mike and i stepped through this. It's true that depending on how you do the selection, you get a subtly different selection range. But the deletion code should bahve the same way for the two different possibilities, so this is definitely my bug. I'll try to fix this during the b3 "quiet period", though I doubt anyone will let me check it in.
setting to future, adding helpwanted
Target Milestone: M19 → Future
*** Bug 71875 has been marked as a duplicate of this bug. ***
moving to moz0.9.2 per triage meeting
Target Milestone: Future → mozilla0.9.2
open the testcase above in composer, highlight all of the text with mouse and press any character key on keyboard. Observe that the first item stays but another empty list item stays there too.
fixed by my changes in 091
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
yeah, veirifed fixed on 0523. Testcase works fine now.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.