another update on the systray and tiny api review

Richard Moore richmoore44 at gmail.com
Mon Mar 23 23:57:39 CET 2009


On Mon, Mar 23, 2009 at 4:03 PM, Aaron J. Seigo <aseigo at kde.org> wrote:
>        bool hidden() const;
>        void setHidden(HideState state);

Asymmetry is evil. If set takes a HideState then get should return it.
Having another method that is a more general helper is ok though.

bool hidden() const;

void setHideState(HideState state);
HideState hideState() const;

This has the advantage of being clearer and also being possible to
make a Q_PROPERTY.

Cheers

Rich.


More information about the Plasma-devel mailing list