----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116693/#review52526 -----------------------------------------------------------
Ah, much better now! Just three things: - 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... - The category headers are still hugging the left border, could they get some padding, too? - 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 > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel