Plasma::PopupApplet improvements

[Chani]

On October 5, 2010 17:17:50 Ivan Cukic wrote:
> On Tuesday, 5. October 2010. 15.37.31 Chani wrote:
> > On October 5, 2010 15:28:14 Ivan Cukic wrote:
> > > Hi all,
> > > 
> > > There are a few things I wanted to ask your for opinions before
> > > implementing them:
> > > 
> > > 1. bool isIconified() const
> > > Introducing this method that would return whether the applet is shown
> > > full, or just as an icon.
> > 
> > so.. would this still return true if it's an icon with the popup
> > currently open?
> 
> It doesn't matter whether it is popped. There is already a method for that.

what's the name of that method?
/me is wondering whether it'll be easy for newbies to tell which is which :)

> 
> > > 2. void setPopupText(const QString & text) and QString popupText()
> > > const This one I'm unsure about whether it is really needed - it would
> > > show an icon beside the text when the applet is in the iconified
> > > state.
> 
> Well, we currently have setPopupIcon(...)

setPopupIconText maybe?