Review Request: Fix Konqueror's MMB click to close tab option

[Commit Hook]

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


This review has been submitted with commit f8309bec7316e6bb6ff90b76cc99c798ff045f6f by Dawit Alemayehu to branch KDE/4.9.

- Commit Hook


On Oct. 8, 2012, 5:26 p.m., Dawit Alemayehu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106615/
> -----------------------------------------------------------
> 
> (Updated Oct. 8, 2012, 5:26 p.m.)
> 
> 
> Review request for KDE Base Apps and David Faure.
> 
> 
> Description
> -------
> 
> The attached patch fixes the bug where clicking on a tab with the MMB while the "Middle-click on a tab to close it" option is checked results in the tab gaining the focus first before being closed instead of being closed in the background. That results in the active tab being changed when a user activates the aforementioned option and clicks on a background tab with the MMB. 
> 
> For the record the reason why this really happens is KTabBar's treatment of the MMB clicks when the setMovable is set to TRUE. In addition to emitting a mouseMiddleClick signal which Konqueror currently relies on to close the tab, it also eats the MMB click event and generates a new LMB click event in its place. Apparently that was done for compatibility sake so that the user can move tabs using either the LMB or MMB.  
> 
> Anyhow, this patch addresses the problem by installing an event filter to intercept the mouse events from the tabbar and handle MMB clicks on its own when the aforementioned option is checked so that we do not receive unnecessary bogus events.
> 
> 
> This addresses bug 264058.
>     http://bugs.kde.org/show_bug.cgi?id=264058
> 
> 
> Diffs
> -----
> 
>   konqueror/src/konqtabs.h 4b6f1f1 
>   konqueror/src/konqtabs.cpp 611659f 
> 
> Diff: http://git.reviewboard.kde.org/r/106615/diff/
> 
> 
> Testing
> -------
> 
> Check/uncheck the "Middle-click on a tab to close it" option and tested the behavior.
> 
> - When the option is unchecked, clicking on a background tab with MMB behaves the same way as it currently does.
> - When the option is checked, clicking on a background tab with MMB will immediately close it without changing the current active tab.
> 
> 
> Thanks,
> 
> Dawit Alemayehu
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20121012/0cb3d7fb/attachment.htm>