[rekonq] Review Request 111610: Fix tabs sometimes not closing using the middle mouse button

[Diego Soenens]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111610/
-----------------------------------------------------------

Review request for rekonq.


Description
-------

If the mouse moves even 1 pixel while the middle mouse button is being pressed/clicked to close a tab it causes KTabBar to not emit mouseMiddleClick() if isMovable() == true.
As a result the tab doesn't get closed and the user has to try again and this can happen a few times in a row if the mouse has a "hard" middle mouse button. Quite annoying :(.

See: http://api.kde.org/4.10-api/kdelibs-apidocs/kdeui/html/ktabbar_8cpp_source.html#l00209
According to the comment it's a "compatibility feature for old middle mouse tab moving" so I'm not exactly sure if this should even be fixed to begin with and whether Rekonq is the place to do it in this case.


Diffs
-----

  src/tabwindow/tabbar.cpp eb062bf 

Diff: http://git.reviewboard.kde.org/r/111610/diff/


Testing
-------

Been running this patch for a few hours. Everything seems to work as intended.


Thanks,

Diego Soenens

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/rekonq/attachments/20130719/0575bc67/attachment.html>