Review Request 126856: Remove text from documentation find widget's buttons

Igor Kushnir igorkuo at gmail.com
Sat Apr 16 19:25:40 UTC 2016


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

(Updated April 16, 2016, 7:25 p.m.)


Review request for KDevelop.


Changes
-------

Make documentation find widget more like KTextEditor's find widget.
This saved both horizontal and vertical space for documentation.

Remove the "Find:" label to save even more horizontal space.
The find widget is recognizable since it is almost the same as the main KTextEditor's find widget.

Screenshots: https://drive.google.com/uc?export=download&id=0B0F-aqLyFk_PSzlielRVZjFfS2s

Tested with the latest master - works the same as with 3-months-old code.


Repository: kdevplatform


Description
-------

Remove text from documentation find widget's buttons

Screenshots: https://drive.google.com/uc?export=download&id=0B0F-aqLyFk_PRWZnS3dLQ3ZHcTQ
Note that with these changes the top Documentation Tool Bar requires more horizontal space than the documentation find widget. I proposed a solution { split the top toolbar in two rows } in https://git.reviewboard.kde.org/r/126685/ (see links to screenshots in my recent comments).


Diffs (updated)
-----

  documentation/documentationfindwidget.cpp 6bf24ab6ecf52d733d13bfae9a459230dab5ca40 
  documentation/documentationfindwidget.ui ce07311f2c89be70c45bf796a76177c0614cabd4 

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


Testing
-------

Built, installed, tested both with and without documentation zoom implementation.


Thanks,

Igor Kushnir

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20160416/297b8d1b/attachment.html>


More information about the KDevelop-devel mailing list