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

[Igor Kushnir]

> On Aug. 2, 2017, 10:39 a.m., Friedrich W. H. Kossebau wrote:
> > 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 :) )

No problem. I assumed that KDevelop mainteiners are bored with the UI changes and so didn't bring this review up.
KDevelop reviews are much more active now than a year ago, which is very good.


- Igor


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


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/7987841b/attachment.html>