48918 - Should be able to transfer items across trees by doubleclicking in Customize Search

Status: NEW

(SeaMonkey :: Search, defect, P3)

Description

[Blake Ross]

Standard windows behavior when you're transferring items between trees (or 
listboxes) is that you can double-click in one tree to send it to the other.  
We should support this in the Customize Search dialog.  For example, if you 
double-click on an item in the left tree, it should be moved to the right, and 
vice versa.

I talked to Brian (Ryner) about this, and he suggested you set onclick for the 
items and then check event.detail for the clickcount. You could also specify 
clickcount= in an XBL handler.  I'm not sure if treeitems fire ondblclick.

Comment 1

[John Unruh]

*** This bug has been marked as a duplicate of 48919 ***

Comment 2

[shrirang khanzode]

verified

Comment 3

[Blake Ross]

This is absolutely not a dup and a closer reading of the bug would have proven 
that.  Please dup-marking and verifying up to the QA contact if you aren't sure.

Comment 4

[Jeffrey Baker]

No reason to get bent out of shape.  It would be helpful, if you are going to
file bugs with almost exactly the same summary, if you would make the summary
fit in the Bugzilla summary box.  That way we can see the whole thing at a
glance.  Otherwise, it's like one of those Sunday comic spot-the-changes games.

Sloppy job on the dup + vrfy, though.

Comment 5

[Marcia Knous [:marcia]]

Netscape nav triage team: as per Matt Fisher's pre-triage recommendation, this 
bug is nsbeta1-.

Comment 6

[Paul Chen]

Marking nsbeta1- bugs as future to get off the radar

Comment 7

[Blake Ross]

reassigning matt's old bugs to default owner.

Comment 8

[Robert Strong (they/them - no direct email)]

Attachment: patch

This patch adds dblclick attributes to the appropriate listitem elements in
search-editor.xul that use the existing AddEngine and RemoveEngine functions in
search-editor.js to handle the event.

Comment 9

[[e-mail]]

Comment on attachment 174864 [details] [diff] [review]
patch

Although generally I'm not a big fan of events on templated items as they
consume more memory.