> 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 > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
