Review Request: Make the uses widget nicer.
Sven Brauch
svenbrauch at gmx.de
Wed Nov 14 21:19:47 UTC 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107296/
-----------------------------------------------------------
(Updated Nov. 14, 2012, 9:19 p.m.)
Review request for KDevelop, KDE Usability and Milian Wolff.
Changes
-------
Sorry, I forgot the mandatory screenshot.
Description
-------
TL;DR: Look at the screenshots below. :)
Make the uses widget nicer.
Main changes include:
* Fix contexts not being clickable (they were clickable but didn't work)
* Remove the blue arrows; all widgets are expanded by default and can
be collapsed with a small link in the text instead. This operation is
not common enough to justify such a huge ugly button.
(The main problem with the button was that it looked ugly, tough)
* Contexts without actual uses are always sorted last.
* Add some nice icons to the list.
* Remove all the ugly frames.
* Make text displaying code use a fixed-width font.
* Make the tooltip use a fixed-width font too, make it smaller (just 2
lines of context instead of 3), and highlight the line with the use
by making it bold-face.
* Re-format and rename all the widget headers to be more readable
and look nicer.
I will add a few comments to the patch to explain what I was doing.
Oh and also, I had that impression for quite a while now, but right now I'm pretty sure the find uses feature is broken in some way. Just look at the screenshots: it found way more occurences in the first screenshot than in the second (which happened after a reparse of some projects). And this has nothing to do with my changes. :)
Diffs
-----
language/duchain/navigation/useswidget.h ec0440e
language/duchain/navigation/useswidget.cpp 8c01e7c
shell/projectcontroller.cpp 6168066
Diff: http://git.reviewboard.kde.org/r/107296/diff/
Testing
-------
Manual testing; no functional changes were done except for the context link fix.
Screenshots (updated)
-----------
Comparsion between the old and the new uses widget.
http://git.reviewboard.kde.org/r/107296/s/821/
The new useswidget after the latest updates
http://git.reviewboard.kde.org/r/107296/s/822/
Third version; the header line updates while uses are being searched
http://git.reviewboard.kde.org/r/107296/s/832/
Thanks,
Sven Brauch
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20121114/d8ba1042/attachment.html>
More information about the KDevelop-devel
mailing list