edit this page and delete Personals section causes stray bullet

VERIFIED FIXED in M1

Status

()

Core
Editor
VERIFIED FIXED
17 years ago
16 years ago

People

(Reporter: sujay, Assigned: Joe Francis)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: EDITORBASE+ [adt2 rtm]; fixinhand; need testing,custrtm-, URL)

Attachments

(1 attachment)

(Reporter)

Description

17 years ago
using 3/8 build

1) launch netscape
2) jump to above page
3) File | Edit Page
4) scroll down and delete all the bullets under Personal, drag mouse from last
bullet all the way upward thru "Personals"
5) delete

now notice under 

Group Issues you have

* None
	o  <--  how did this bullet get there? it was not there
originally 		                before step 5) above
(Reporter)

Comment 1

17 years ago
changing summary
Summary: edit this page and enter on last bullet causes bullet to move left → edit this page and delete Personals section causes stray bullet
(Assignee)

Comment 2

17 years ago
Sadly, the word "Personals" does not occur anywhere in the document
http://jazz/users/sujay/publish/marsqa.html.  Did you mean to link a different
document?

Marking invalid so that Sujay will see this.
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → INVALID
(Reporter)

Comment 3

17 years ago
Joe, I fixed the URL...you shoud be able to replicate this problem now...

REOPEN.
Status: RESOLVED → REOPENED
Keywords: nsbeta1
Resolution: INVALID → ---
Whiteboard: EDITORBASE

Updated

17 years ago
Keywords: nsbeta1 → nsbeta1+
Whiteboard: EDITORBASE → EDITORBASE+
Target Milestone: --- → mozilla1.0.1

Updated

17 years ago
Whiteboard: EDITORBASE+ → EDITORBASE+ [adt2]
Clearing EDITORBASE+ and nsbeta1 for re-consideration since the fix has been
determined to be both difficult and risky.
Keywords: nsbeta1+ → nsbeta1
Whiteboard: EDITORBASE+ [adt2] → EDITORBASE [adt2]

Updated

17 years ago
Keywords: nsbeta1 → nsbeta1-

Comment 5

17 years ago
Joe, what is the risk, how long will this take to fix? This really looks like 
the kind of thing we want for EDITORBASE.

Updated

17 years ago
Component: Editor: Composer → Editor: Core
editorbase triage: Marking EDITORBASE+, nsbeta1+, [adt1]. This is a very common
operation.
Keywords: nsbeta1- → nsbeta1+
Whiteboard: EDITORBASE [adt2] → EDITORBASE+ [adt1]
Simple test case:

Create a new document.
Add a list item
Add another list item and indent it so you have the following

 *item1
    *item2

Select item2 and item1 by dragging from item2 over item1.

Hit the delete key. You will see the following

 *
    *

Only the item content seems to have been selected.
If you do a select all it works as expected.

Selection issue?

Joe: Is this a selection issue? Should this be reassigned to mjudge?


Status: REOPENED → NEW
(Assignee)

Comment 8

17 years ago
Kevin: the test case you mention in comment #7 is a known issue.  There may be a 
bug agisnt me for that already.  

Comment 9

17 years ago
marking [adt1 rtm].  This is not a beta stop ship bug.  Let's get this in for rtm.
Whiteboard: EDITORBASE+ [adt1] → EDITORBASE+ [adt1 rtm]
(Assignee)

Comment 10

16 years ago
I think I know what has to happen here but it is not something we would want on
the branch without a lot of testing.  trying to finish up a patch soon so folks
can try it out.
Status: NEW → ASSIGNED
(Reporter)

Comment 11

16 years ago
If this is something we don't want on the branch then I think
we should minus this unless Joe can do a lot of unit testing
on this fix.

Updated

16 years ago
Blocks: 143047
(Assignee)

Comment 12

16 years ago
Created attachment 83717 [details] [diff] [review]
patch to nsHTMLEditRules.{h,cpp}

I've been tweaking this for a long time now, and I think it is ready for more
eyes.  This changes the behavior of deletion when you have a non-collapsed
selection.  We should get some feedback on if this is good or bad.

Updated

16 years ago
Target Milestone: mozilla1.0.1 → mozilla1.0
(Assignee)

Updated

16 years ago
Whiteboard: EDITORBASE+ [adt1 rtm] → EDITORBASE+ [adt1 rtm]; fixinhand; need r=,sr=,testing

Updated

16 years ago
Whiteboard: EDITORBASE+ [adt1 rtm]; fixinhand; need r=,sr=,testing → EDITORBASE+ [adt1 rtm]; fixinhand; need r=,sr=,testing,custrtm-

Comment 13

16 years ago
Lowering impact to [ADT2 RTM], per ADT triage.
Whiteboard: EDITORBASE+ [adt1 rtm]; fixinhand; need r=,sr=,testing,custrtm- → EDITORBASE+ [adt2 rtm]; fixinhand; need r=,sr=,testing,custrtm-
(Assignee)

Comment 14

16 years ago
The trunk is the wave of the future!
Target Milestone: mozilla1.0 → mozilla1.1beta
nsbeta1-. The fix is too risky for the 1.0 branch.
Keywords: nsbeta1+ → nsbeta1-
(Assignee)

Comment 16

16 years ago
The days of having a half dozen milestones out in front of us to divide bugs 
between seem to be gone, though I dont know why.  Lumping everything together as 
far out as I can.  I'll pull back things that I am working on as I go.
Target Milestone: mozilla1.1beta → mozilla1.2beta
(Assignee)

Updated

16 years ago
Target Milestone: mozilla1.2beta → M1

Updated

16 years ago
Keywords: nsbeta1- → nsbeta1+

Updated

16 years ago
Target Milestone: M1 → mozilla1.2alpha
Attachment #83717 - Flags: review+
Comment on attachment 83717 [details] [diff] [review]
patch to nsHTMLEditRules.{h,cpp}

r=glazman

Comment 19

16 years ago
Comment on attachment 83717 [details] [diff] [review]
patch to nsHTMLEditRules.{h,cpp}

sr=kin@netscape.com
Attachment #83717 - Flags: superreview+

Updated

16 years ago
Whiteboard: EDITORBASE+ [adt2 rtm]; fixinhand; need r=,sr=,testing,custrtm- → EDITORBASE+ [adt2 rtm]; fixinhand; need testing,custrtm-
(Assignee)

Comment 20

16 years ago
fix landed on trunk
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago16 years ago
Resolution: --- → FIXED
(Assignee)

Comment 21

16 years ago
fix does not seem to working on trunk.  investigating...
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Target Milestone: mozilla1.2alpha → M1
(Assignee)

Comment 22

16 years ago
ok, really fixed now.
Status: REOPENED → RESOLVED
Last Resolved: 16 years ago16 years ago
Resolution: --- → FIXED
(Reporter)

Comment 23

16 years ago
verified in 9/5 trunk build.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.