Hi, Looks good, I like it :). However, I don't like that there're different ways of displaying this add/remove thingy in KDE, e.g. Amarok - http://images.maketecheasier.com/2008/12/amarok-dynamic-playlist.jpg Dolphin search - http://2.bp.blogspot.com/_zDPqioLuxns/Sv6j1Y548ZI/AAAAAAAAAHg/XLSyabEI33w/s1600-h/searching0.png
I suggest that you 1. Resize the width of the buttons (configure, about, remove) to be consistent with other dialogs, see "All Effects" in "Desktop Effects" module for example 2. Change actions/edit-delete.png icon to actions/list-remove.png for remove button 3. Move remove buttons to the left-most 4. Not sure about this; make "Add Trigger" button fill two cells of the layout, below a remove button and trigger button ("Right-Button" in your screenshoot) With best regards, Hans Chen On Mon, Nov 23, 2009 at 06:47, Hugo Pereira Da Costa <h...@oxygen-icons.org>wrote: > Hi Chani, > just a naive question, why do you use a "sunken" frame to display the > triggers ? As far as I know this is the only place where things are > presented this way (usually config have either Tabs, group boxes, or are > plain flat). Is there a specific reason here ? > > so I've been working on my plugin config UI again, based on suggestions from > aaron. it's not 100% done but it's most of the way there now. :) > > I've rearrranged some things, and added some > stuff...http://chani.ca/sshots/mousplugins.png > in retrospect, using reviewboard might've been a good idea. > anyways, I'm not happy with the new strings at the bottom there. not happy at > all. any suggestions? > > oh yeah, and remember, string (and feature) freeze is wednesday. :) > > > > > _______________________________________________ > Plasma-devel mailing > listplasma-de...@kde.orghttps://mail.kde.org/mailman/listinfo/plasma-devel > > > > _______________________________________________ > Plasma-devel mailing list > Plasma-devel@kde.org > https://mail.kde.org/mailman/listinfo/plasma-devel > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel