Review Request 116693: Improve layout of Effect View

Thomas Pfeiffer colomar at autistici.org
Mon Mar 10 13:03:07 UTC 2014


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


Much better now, thanks for fixing these issues so quickly!
Just three small things left:


File Attachment: Effects View together with the QtQuick Controls text field - kwineffects.png
<https://git.reviewboard.kde.org//r/116693/#fcomment191>
    The search field still says "Search Plugins" but should say "Search Effects" (for consistency) or better still just "Search" (because what else should it search for?). Reminds me that I really have to write that Search HIG to standardize such things...


File Attachment: Effects View together with the QtQuick Controls text field - kwineffects.png
<https://git.reviewboard.kde.org//r/116693/#fcomment192>
    The category headers are still hugging the left border, could they get some padding, too? 


File Attachment: Effects View together with the QtQuick Controls text field - kwineffects.png
<https://git.reviewboard.kde.org//r/116693/#fcomment193>
    The effect names were bold in the Plasma 1 version and they should be bold here as well (sorry for not noticing that before)

- Thomas Pfeiffer


On March 10, 2014, 9:48 a.m., Martin Gräßlin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/116693/
> -----------------------------------------------------------
> 
> (Updated March 10, 2014, 9:48 a.m.)
> 
> 
> Review request for kwin, Plasma, David Edmundson, Thomas Pfeiffer, and Antonis Tsiapaliokas.
> 
> 
> Repository: kwin-compositing-kcm
> 
> 
> Description
> -------
> 
> Improve layout of Effect View
> 
> * use frame in the scroll area
> * remove needless anchoring for an Effect
> * use one RowLayout for one Effect row
> * add a left and right padding using the normal spacing
> * Use a ColumnLayout for the center element consisting of
>   ** name
>   ** description
>   ** (info)
>   ** (video)
> 
> 
> Diffs
> -----
> 
>   qml/Effect.qml 62ec89c83b02f14facd74eabaf4f4ba2cb0cab31 
>   qml/EffectView.qml c1a66dfa41409ba04c4cf4ddded3d95bb4bfaef2 
>   qml/Video.qml PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/116693/diff/
> 
> 
> Testing
> -------
> 
> 
> File Attachments
> ----------------
> 
> Effects View together with the QtQuick Controls text field
>   https://git.reviewboard.kde.org/media/uploaded/files/2014/03/10/3dd02cd5-e489-45ae-a66a-92a62a4ef5e3__kwineffects.png
> 
> 
> Thanks,
> 
> Martin Gräßlin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140310/f5d05530/attachment.html>


More information about the Plasma-devel mailing list