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 620381 - [StEntry] Consider children when setting hover state
[StEntry] Consider children when setting hover state
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-06-02 17:14 UTC by Florian Müllner
Modified: 2010-06-02 21:23 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
[StEntry] Consider children when setting hover state (1.74 KB, patch)
2010-06-02 17:14 UTC, Florian Müllner
committed Details | Review

Description Florian Müllner 2010-06-02 17:14:33 UTC
See attached patch - we might want to make actor_contains() public as st_widget_contains() to avoid code duplication between StWidget and StEntry.
Comment 1 Florian Müllner 2010-06-02 17:14:37 UTC
Created attachment 162561 [details] [review]
[StEntry] Consider children when setting hover state

When hovering over the entry's ClutterText, the entry itself
currently receives a leave event and updates the hover state.
Apply the same logic as StWidget itself to treat children as
part of the entry.
Comment 2 Owen Taylor 2010-06-02 19:55:02 UTC
Review of attachment 162561 [details] [review]:

Looks good
Comment 3 Florian Müllner 2010-06-02 21:23:16 UTC
Attachment 162561 [details] pushed as cc16323 - [StEntry] Consider children when setting hover state