After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 593503 - ShellButtonBox: Use default handlers, not self-connections
ShellButtonBox: Use default handlers, not self-connections
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks: 593471
 
 
Reported: 2009-08-29 17:41 UTC by Owen Taylor
Modified: 2009-08-29 19:33 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
ShellButtonBox: Use default handlers, not self-connections (4.98 KB, patch)
2009-08-29 17:41 UTC, Owen Taylor
accepted-commit_now Details | Review

Description Owen Taylor 2009-08-29 17:41:32 UTC
There's seldom a good justification for connecting to signals on
yourself rather than using the default handler slots in the class.

But in particular using the default handler slots means that
an application can connect to ::button-press-event and get in
before the default handling, to implement a button that does
something on press.
Comment 1 Owen Taylor 2009-08-29 17:41:34 UTC
Created attachment 141992 [details] [review]
ShellButtonBox: Use default handlers, not self-connections
Comment 2 Colin Walters 2009-08-29 17:44:40 UTC
Comment on attachment 141992 [details] [review]
ShellButtonBox: Use default handlers, not self-connections

Yep; sorry, was translating from a half-done JS version I had written before deciding properties with notification were the way to go.
Comment 3 Owen Taylor 2009-08-29 19:33:59 UTC
Pushed