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