Closed
Bug 4057
Opened 26 years ago
Closed 26 years ago
nsRangeList::Clear() not resetting focus and anchor nodes
Categories
(Core :: DOM: Selection, defect, P3)
Core
DOM: Selection
Tracking
()
VERIFIED
FIXED
M4
People
(Reporter: mozeditor, Assigned: mozeditor)
Details
(Whiteboard: 3-28 waiting on reporter/owner to verify)
Clear() is just chucking out the dom ranges that represent the selection, but it
is not also clearing out the frame level selection information.
There are three possibilities here:
1) leave it like it is
2) have it clear out the presentation shells concept of focus frame and anchor
frame
3) do 2) plus actually walk the frame list and unset any selection bits in the
frames.
I'm examining the usage and intent of Clear() to determine which of these is
correct.
Assignee | ||
Updated•26 years ago
|
Status: NEW → ASSIGNED
Target Milestone: M4
Assignee | ||
Updated•26 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Summary: nsRangeList::Clear() not resetting focus and anchor frames → nsRangeList::Clear() not resetting focus and anchor nodes
Assignee | ||
Comment 1•26 years ago
|
||
ooops, misunderstood Steve at first. He meant that Clear() wasn't setting focus
and anchor NODES, not frames. Fixed.
Updated•26 years ago
|
Whiteboard: waiting on reporter/owner to verify
Comment 2•26 years ago
|
||
If the reporter or owner could verify this bug and make it go away for all of us I would be much obliged. thank you
Updated•26 years ago
|
Whiteboard: waiting on reporter/owner to verify → 3-28 waiting on reporter/owner to verify
Assignee | ||
Comment 3•26 years ago
|
||
I've asked Steve Clark to verify this. Although I made the "fix", I was never
able to see a problem in the first place, so I want Steve to verify.
you don't see any problem because I had worked around it. I looked at the code
you checked in and it seems fine.
Updated•26 years ago
|
Status: RESOLVED → VERIFIED
Comment 5•26 years ago
|
||
i'm taking the above statements to mean this bug fix is as verified as its ever going to get. marking VERIFIED
Per a request from Selection and Search component eng (mjudge) and qa (elig),
moving all "Selection and Search" bugs to new "Selection" component. Original
"Selection and Search" component will be retired.
You need to log in
before you can comment on or make changes to this bug.
Description
•