Closed Bug 43417 Opened 24 years ago Closed 24 years ago

Dragging a tree item doesn't first select the cell

Categories

(Core :: XUL, defect, P3)

defect

Tracking

()

VERIFIED FIXED

People

(Reporter: mikepinkerton, Assigned: bryner)

Details

(Whiteboard: [nsbeta2+][dogfood-])

Attachments

(1 file)

If you select an message in the mail thread pane, then click and drag another to 
a folder, you'll end up moving the first item because the mail trees use 
selection and the item you dragged is never selected.

this can be very very confusing as you can end up moving something different than 
what you thought (on macos, you even _see_ the object you're dragging but you are 
operating on something else).
sorry hyatt, this is dogfood.
Keywords: dogfood
Putting on [nsbeta2+][dogfood-] radar.  Does not need a fix ASAP for daily work, 
but we should fix this for beta2. 
Whiteboard: [nsbeta2+][dogfood-]
Hm, it looks like this can be fixed by changing the selectElement handler in
treeBindings.xml from a click handler to a mousedown handler.  Will that work,
or is there more to it?
There are subtleties.  Sometimes selection happens on a mousedown, and sometimes 
it happens on a mouseup.  We should talk about it, and then produce a patch.
Target Milestone: --- → M18
Reassigning to myself.  Keeping hyatt on CC in case he wants to add input.
Assignee: hyatt → bryner
Status: NEW → ASSIGNED
Attached patch patchSplinter Review
Ok, the patch I just attached should take care of a lot of problems involving
multiple selection, including this one.
Fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
verified fixed. The cell is selected at the beginning of the drag 
(onmousedown). mac/win32/linux 20000712nn builds -- tested in mail, bookmarks. 

[Although, on the drop end of this action, there is a crash -- bug 45361]
Status: RESOLVED → VERIFIED
Adding keyword to bugs which already show a nsbeta2 triage value in the status 
whiteboard so the queries don't get screwed up.
Keywords: nsbeta2
Component: XP Toolkit/Widgets: Trees → XUL
QA Contact: jrgmorrison → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: