Review Request: Rework layout of widget explorer and activity manager in vertical mode

[Aurélien Gâteau]

> On June 25, 2012, 9:32 a.m., Thomas Pfeiffer wrote:
> > What do you mean by "Fix cropped list items"? They are still cropped and look pretty much the same to me. 
> > And actually fixing them (e.g. by line-wrapping instead of cropping since there usually is enough space for an extra line) would be quite important.
> > Other than that it looks fine to me and definitely like an improvement.

I am talking about the item borders, not the item text. I agree item description text should wrap, I am actually doing this as well.


- Aurélien


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105329/#review15104
-----------------------------------------------------------


On June 25, 2012, 8:23 a.m., Aurélien Gâteau wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105329/
> -----------------------------------------------------------
> 
> (Updated June 25, 2012, 8:23 a.m.)
> 
> 
> Review request for Plasma, KDE Usability and Marco Martin.
> 
> 
> Description
> -------
> 
> I made a few changes to make widget explorer nicer in vertical mode
> 
> - Fix cropped list items.
> - Make buttons use full width.
> - Move "Get new widgets" and action buttons to the bottom so that:
>   - different label alignments do not look odd (category button vs others)
>   - category button is closer to the list it controls
> 
> Additionally it would be much better if tooltips could appear on the right (or left) of the current list item instead of on top of it because it easily gets in the way, but I don't know how to do this. Any idea? An alternative would be to just drop the tooltip, it currently does more harm than good imo.
> 
> There is also something weird going on with the backgrounds: the "after" screenshots use a opaque background, but that is not part of the patch and I don't understand what makes the code pick the opaque or transparent background. It should be fixed to use the opaque background, again, any idea where that bug could be?
> 
> 
> Diffs
> -----
> 
>   libs/plasmagenericshell/widgetsexplorer/package/contents/ui/main.qml 4beecec 
>   plasma/desktop/shell/activitymanager/package/contents/ui/main.qml 1886529 
> 
> Diff: http://git.reviewboard.kde.org/r/105329/diff/
> 
> 
> Testing
> -------
> 
> - Looks better in vertical mode.
> - No regression in horizontal mode.
> 
> 
> Screenshots
> -----------
> 
> before-after
>   http://git.reviewboard.kde.org/r/105329/s/610/
> 
> 
> Thanks,
> 
> Aurélien Gâteau
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20120625/672e268b/attachment-0001.html>