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

[Friedrich W. H. Kossebau]

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



Sadly this patch was forgotten. And only now rewritten as part of https://phabricator.kde.org/D7061 (and almost the same as in last version here :) ).
So Igor, could you please close this request as discarded, to help cleaning up remaining requests here on reviewboard? Reviewboard going to be closed soon finally in favour of phabricator, myself I have not even ever looked for kdevelop patches here.
Any remaining discussion on the things changed in this patch should be done as part of that never request now.

(And lesson to learn: in the future, best be less shy and poke maintainers again and again, especially if there already is positive feedback. as usual people only really care about their own stuff and forget about the rest, as real life also brings lots of new stuff to attention all the time, so no bad intentions :) )

- Friedrich W. H. Kossebau


On April 24, 2016, 2:53 p.m., Igor Kushnir wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126856/
> -----------------------------------------------------------
> 
> (Updated April 24, 2016, 2:53 p.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> 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
> -----
> 
>   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/20170802/dc082a4f/attachment-0001.html>