[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-kimageshop
Subject:    Re: [calligra] krita/plugins/extensions/dockers: Merge
From:       Boudewijn Rempt <boud () valdyas ! org>
Date:       2011-08-09 5:37:01
Message-ID: 201108090737.01483.boud () valdyas ! org
[Download RAW message or body]

On Monday 08 August 2011 Aug, Silvio Heinrich wrote:
> Am 07.08.2011 11:31, schrieb Boudewijn Rempt:
> > On Saturday 06 August 2011 Aug, Silvio Heinrich wrote:
> > > Git commit 1e523a6e50ac55a6e5a07963e493e8878593cf2c by Silvio Heinrich.
> > > Committed on 06/08/2011 at 16:14.
> > > Pushed by heinrich into branch 'master'.
> > > 
> > > Merge remote-tracking branch 'origin/krita-imagedocker-silvioheinrich'
> > > 
> > > M  +1    -0    krita/plugins/extensions/dockers/CMakeLists.txt
> > > 
> > > http://commits.kde.org/calligra/1e523a6e50ac55a6e5a07963e493e8878593cf2c
> > > 
> > 
> > Cool stuff! Couple of small remarks:
> > 
> > I think that for marking images as selected, we should use the standard KDE \
> > selection markers (-, + as labels) and the outline) instead of the raster \
> > overlaid. 
> > For the color docker, it would be better to use the wrench icon we have \
> > everywhere for settings instead of the "pref" label. 
> 
> what are the icon names for those two?

for the settings, "configure", for the plus/minus: list-add and list-remove.

-- 
Boudewijn Rempt
http://www.valdyas.org, http://www.krita.org, http://www.boudewijnrempt.nl
_______________________________________________
kimageshop mailing list
kimageshop@kde.org
https://mail.kde.org/mailman/listinfo/kimageshop


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic