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 646915 - panel: provide a consistent way to refer to panel items from lg
panel: provide a consistent way to refer to panel items from lg
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:
 
 
Reported: 2011-04-06 13:32 UTC by Dan Winship
Modified: 2011-04-26 12:22 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
panel: provide a consistent way to refer to panel items from lg (3.29 KB, patch)
2011-04-06 13:32 UTC, Dan Winship
committed Details | Review

Description Dan Winship 2011-04-06 13:32:40 UTC
There is currently no simple way to refer to a status area menu from
within lg; the easiest way to get to it currently is to use the
picker, click the menu, and get the actor's _delegate. This is
annoying when testing, and also annoying when trying to help other
people debug (eg, bug 646902 comment 2).

(The activities button is currently available as "Main.panel.button",
because main.js needs access to it, but I added Main.panel._activities
as well for consistency.)
Comment 1 Dan Winship 2011-04-06 13:32:41 UTC
Created attachment 185306 [details] [review]
panel: provide a consistent way to refer to panel items from lg

Rename a few Panel fields and add a _statusArea object pointing to the
status area PanelButton objects. You can now address each item
consistently from lg:

    Main.panel._activities
    Main.panel._appMenu
    Main.panel._dateMenu
    Main.panel._statusArea.a11y
    Main.panel._statusArea.volume ...
    Main.panel._userMenu
Comment 2 Colin Walters 2011-04-07 13:52:22 UTC
Review of attachment 185306 [details] [review]:

This looks fine.
Comment 3 Dan Winship 2011-04-26 12:22:12 UTC
Attachment 185306 [details] pushed as 9d5906d - panel: provide a consistent way to refer to panel items from lg