[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: Re: [Panel-devel] KDE/kdebase/workspace
From: Laurent Montel <montel () kde ! org>
Date: 2007-07-28 22:16:29
Message-ID: 200707290016.30229.montel () kde ! org
[Download RAW message or body]
On Friday 27 July 2007 23:10:47 Aaron J. Seigo wrote:
> On Thursday 26 July 2007, Laurent Montel wrote:
> > SVN commit 693020 by mlaurent:
> >
> > Allow to lock applet position
> >
> >
> > M +19 -0 libs/plasma/applet.cpp
> > M +9 -0 libs/plasma/applet.h
> > M +6 -0 plasma/plasma/desktopview.cpp
Hi,
> besides the fact that this commit does not follow the coding guidelines
> (e.g. uses tabs instead of spaces), was not peer reviewed (that i know of?)
> and introduces new public API without apidox ........
>
> ... why do we want to lock the position of an applet? yes, i know SK did
> this, but i don't see the use case for it in plasma. can you provide a
> reason why we should offer "Lock" in the context menu?
Because when we have several applets on desktop we want to put them in a
specific position. We won't want to move it by accident.
and if we want to lock them we must have a menu for it.
> the patch is also broken in numerous ways, primarily surrounding kiosk and
> immutability.
Why ?
Menu entry is put after test if immutability and kiosk option.
> *if* we offer per-applet locking, it should simply call
> setImmutable.
_______________________________________________
Panel-devel mailing list
Panel-devel@kde.org
https://mail.kde.org/mailman/listinfo/panel-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic