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 611211 - [lookingGlass] Add style information to the picker
[lookingGlass] Add style information to the picker
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: 2010-02-26 18:23 UTC by Florian Müllner
Modified: 2010-02-26 22:27 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
[lookingGlass] Add style information to the picker (1.70 KB, patch)
2010-02-26 18:23 UTC, Florian Müllner
committed Details | Review

Description Florian Müllner 2010-02-26 18:23:21 UTC
Originally not intended for inclusion, but I found myself using it too much to not at least propose it:

Display style id and class when hovering over StWidgets in looking glass pick-mode ...
Comment 1 Florian Müllner 2010-02-26 18:23:38 UTC
Created attachment 154780 [details] [review]
[lookingGlass] Add style information to the picker

Currently the picker displays information on the hovered actor's
position and the object itself. Adding style id and class of
StWidgets to the displayed information can be very helpful for
identifying a particular actor or code segment.
Comment 2 Colin Walters 2010-02-26 18:57:22 UTC
Review of attachment 154780 [details] [review]:

Looks good but a specific tab with style information would be cooler.  Feel free to commit this though, we can add the latter later.
Comment 3 Colin Walters 2010-02-26 18:58:21 UTC
Review of attachment 154780 [details] [review]:

Actually nevermind, there's cases for both actually.  We don't want to make the hover information *too* big though so we should limit it to not more than this.  Or we'll need intelligence to move the hover when the mouse pointer enters it.
Comment 4 Florian Müllner 2010-02-26 22:26:57 UTC
Attachment 154780 [details] pushed as d7075d9 - [lookingGlass] Add style information to the picker