[Panel-devel] Plasma::Icon highlighting issues

[Aaron J. Seigo]

On Friday 14 December 2007, Jason Stubbs wrote:
> On Friday 14 December 2007 01:40:33 Aaron J. Seigo wrote:
> > > * Remove the pressed(bool) signal as kickoff is the only code using it
> > > and the next patch changes that
> >
> > it's there for completeness. think of menus that use this button. it
> > needs to stay.
>
> Is pressed(false) ever relevant? I'd at least like to remove that.

pressed(false) should be emitted on button up. so through a full interaction 
the signals would go like this:

	pressed(true)
	clicked()
	pressed(false)

or like this:

	pressed(true)
	pressed(false)

in the case when the use presses, moves off the button, releases the mouse.

> > cool. there's a BR somewhere for this as well in the plasma component.
>
> That's why I'm working on it. The war with bugzilla can never be won, but
> lower bug numbers makes things easier for everybody. :)

+1 with cookies on top

-- 
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 Trolltech
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/panel-devel/attachments/20071214/965bae90/attachment-0001.pgp