overhaul of some kcm appearance
Aaron J. Seigo
aseigo at kde.org
Wed Mar 11 23:13:28 GMT 2009
On Tuesday 10 March 2009, Oswald Buddenhagen wrote:
> On Tue, Mar 10, 2009 at 05:22:40PM -0600, Aaron J. Seigo wrote:
> > On Monday 09 March 2009, Oswald Buddenhagen wrote:
> > > On Wed, Mar 04, 2009 at 11:37:23PM +0100, Marco Martin wrote:
> > > > is it ok? comments?
> > >
> > > i protest the idea of pulling libplasma into everything.
> > not that it is anyone's goal to "pull libplasma into everything", but
> > what is your objection to using it where it makes sense?
> pulling a 2 meg lib (which again pulls something like 40 more libs of
> *very* varying sizes) for one small class is where "makes sense" does
> not apply by my standards.
in this case we're talking about using it in kcm's that are certainly part of
the KDE4 workspace. as such, these libraries will already be loaded and in use
by other apps, and it's kcm's not some small utility where this would have a
long term and meaningful impact on usage.
i wouldn't use it in ksnapshot, for instance ;)
> > > the background configurer for plasma can handle Plasma::Wallpaper
> > > itself.
> > so make every plasma based application (not just the one dialog) more
> > complex instead of just using it where it makes sense? that isn't
> > sensible.
> if you can come up with sufficiently many plasma apps to justify it (by
> whatever standards you may want to apply), you are free to add a derived
> class to libplasma for convenience.
plasma-mid, plasma-overlay (screensaver) .. perhaps one day even kdm-plasma ;)
> while "plasma everywhere" may seem self-evident to you, not all of kde
oh, don't worry... "plasma everywhere" is _not_ the goal nor purpose.
> is or even should be somehow plasma-related.
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA EE75 D6B7 2EB1 A7F1 DB43
KDE core developer sponsored by Qt Software
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 197 bytes
Desc: This is a digitally signed message part.
More information about the kde-core-devel