[Panel-devel] Plasma::Icon highlighting issues

[Jason Stubbs]

On Saturday 15 December 2007 02:32:51 Aaron J. Seigo wrote:
> 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.

After some thinking, I found a theoretical use case that needs a 
pressed(false) and it would require it regardless of whether clicked() 
occurred, so I've come to agree with the above. :)

I've attached new patches which should address all the other issues too.

-- 
Jason Stubbs
-------------- next part --------------
A non-text attachment was scrubbed...
Name: icon-highlighting.patch
Type: text/x-diff
Size: 11572 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/panel-devel/attachments/20071215/5b95bbb3/attachment-0002.bin 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kickoff-highlighting.patch
Type: text/x-diff
Size: 4054 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/panel-devel/attachments/20071215/5b95bbb3/attachment-0003.bin