Review Request 116693: Improve layout of Effect View

[Martin Gräßlin]

> On March 10, 2014, 11:15 a.m., David Edmundson wrote:
> > qml/Video.qml, line 51
> > <https://git.reviewboard.kde.org/r/116693/diff/1/?file=253176#file253176line51>
> >
> >     can this become:
> >     
> >     visible: videoItem.playbackState != MediaPlayer.StoppedState
> >     
> >     and then remove the one line in onClicked and the connection?

nope, doesn't work. Button doesn't become visible. The MediaPlayer states are a little bit strange and tricky


- Martin


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


On March 10, 2014, 10: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, 10: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/92fa78a2/attachment-0001.html>