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 744307 - Don't allow clicking location entry security indicator when location entry is not visible
Don't allow clicking location entry security indicator when location entry is...
Status: RESOLVED OBSOLETE
Product: epiphany
Classification: Core
Component: Interface
3.14.x (obsolete)
Other Linux
: Normal minor
: ---
Assigned To: Michael Catanzaro
Epiphany Maintainers
Depends on:
Blocks: 755518
 
 
Reported: 2015-02-11 04:57 UTC by Michael Catanzaro
Modified: 2016-09-11 03:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
screenshot (133.29 KB, image/png)
2015-02-11 04:57 UTC, Michael Catanzaro
  Details
screencast demonstrating issue (480.67 KB, application/octet-stream)
2016-01-15 23:06 UTC, Michael Catanzaro
  Details
title-box: Do not propagate button press events in title mode (1.84 KB, patch)
2016-02-26 03:39 UTC, Michael Catanzaro
committed Details | Review

Description Michael Catanzaro 2015-02-11 04:57:47 UTC
Created attachment 296556 [details]
screenshot

Don't allow clicking location entry security indicator when location entry is not visible. See the screenshot; this should not happen.
Comment 1 Michael Catanzaro 2015-04-09 18:19:35 UTC
This also happens for the favicon: you can start a drag with the favicon even when the location entry is not visible, by correctly guessing its position underneath the title box!
Comment 2 Yosef Or Boczko 2015-04-11 20:24:19 UTC
I dosn't happen for me here.
Comment 3 Michael Catanzaro 2016-01-15 23:06:03 UTC
Created attachment 319158 [details]
screencast demonstrating issue
Comment 4 Michael Catanzaro 2016-02-26 03:39:29 UTC
Created attachment 322433 [details] [review]
title-box: Do not propagate button press events in title mode

This is a speculative fix for the problem where clicking at just the
right spot in the header bar opens the security popover via the security
indicator in the hidden location entry. I haven't found any good way to
reproduce this problem, unfortunately, but it happens regularly and I
think there's a good chance this will fix it.

This works because there's nothing inside the title box that users
should be allowed to click on when in title mode, except for the
security indicator, which is handled in this very function.
Comment 5 Carlos Garcia Campos 2016-02-26 06:41:53 UTC
Comment on attachment 322433 [details] [review]
title-box: Do not propagate button press events in title mode

It makes sense, yes. Thanks!
Comment 6 Michael Catanzaro 2016-02-26 14:57:15 UTC
Attachment 322433 [details] pushed as 25df867 - title-box: Do not propagate button press events in title mode
Comment 7 Michael Catanzaro 2016-02-28 19:50:56 UTC
For the record, I've also sometimes accidentally dragged the favicon out of the title box when in title mode, though that is harder. Almost never have I been able to reproduce any of this intentionally, so I can only hope it's fixed.
Comment 8 Michael Catanzaro 2016-03-23 14:16:18 UTC
Didn't work :(