Review Request 127815: [KFind] Disable only tab contents not the entire tab bar

Kai Uwe Broulik kde at privat.broulik.de
Mon Jun 6 19:09:30 BST 2016


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

(Updated June 6, 2016, 6:09 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Base Apps and KDE Usability.


Changes
-------

Submitted with commit 7d94f8ce772b04fb26d3d8c03c71d7770fcba45c by Kai Uwe Broulik to branch frameworks.


Repository: kde-baseapps


Description
-------

While searching the entire tab widget got disabled so you couldn't even switch tabs to see what you're actually looking for, eg. if you started searching while on the "Properties" tab you couldn't switch to the "Name/Location" tab.

This patch disables only the tab contents while still allowing to switch tabs.


Diffs
-----

  kfind/kftabdlg.cpp 3de1bfe 

Diff: https://git.reviewboard.kde.org/r/127815/diff/


Testing
-------

Works. I was surprised to find kde-baseapps master still being 4.x-based.


File Attachments
----------------

Before
  https://git.reviewboard.kde.org/media/uploaded/files/2016/05/02/7dfbde66-ded8-4881-b7d4-86ba253afae4__kfindbefore.png
After
  https://git.reviewboard.kde.org/media/uploaded/files/2016/05/02/d7d22fb6-c326-41c3-836a-4fa3bce8cc76__kfindafter.png


Thanks,

Kai Uwe Broulik

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


More information about the kde-core-devel mailing list