Tab closure on mouse down

Bug #721102 reported by Robert K
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
DC++
Fix Released
Undecided
Unassigned

Bug Description

Closure of a tab is done with middle mouse button on "mouse down". Most other applications with tabs close them on "mouse up" and it 'feels' better that way IMO.

Revision history for this message
Szabolcs Molnár (fleet) wrote :

Now that you mentioned, I tried it with some random tabbed applications (Chrome, Firefox, Notepad++), and you're right. They really work that way, so I support the change in DC++ too.

Changed in dcplusplus:
status: New → Confirmed
Revision history for this message
iceman50 (bdcdevel) wrote :

Made a small patch to change =)

Revision history for this message
poy (poy) wrote :

good start. :) but this will need an additional verification to make sure that the tab that received the "mouse down" event is the same as the one on "mouse up". eg i start middle-clicking on tab1, then drag the mouse to tab2; no tab should be closed in that case.

what about shift + left-click? should that also be changed to take effect only after the mouse has been released?

Revision history for this message
Szabolcs Molnár (fleet) wrote :

"what about shift + left-click? should that also be changed to take effect only after the mouse has been released?"

Probably yes

Revision history for this message
iceman50 (bdcdevel) wrote :

I think that would probably be the best way, I was actually thinking about that to myself (in regards to making sure that the tab that receives the mouse click event is the tab that gets it)

Revision history for this message
poy (poy) wrote :

in rev 2660.

Changed in dcplusplus:
status: Confirmed → Fix Committed
Revision history for this message
poy (poy) wrote :

Fixed in DC++ 0.790.

Changed in dcplusplus:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.