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 441419 - "go" button's behaviour
"go" button's behaviour
Status: RESOLVED FIXED
Product: epiphany
Classification: Core
Component: Interface
2.19.x
Other Linux
: Normal normal
: ---
Assigned To: Epiphany Maintainers
Marco Pesenti Gritti
Depends on:
Blocks:
 
 
Reported: 2007-05-26 14:38 UTC by Sebastien Bacher
Modified: 2007-06-05 20:47 UTC
See Also:
GNOME target: ---
GNOME version: 2.19/2.20


Attachments
Solution for bug 441419. (333 bytes, patch)
2007-05-28 20:15 UTC, Sebastiaan Samyn
committed Details | Review

Description Sebastien Bacher 2007-05-26 14:38:56 UTC
The bug has been opened on https://bugs.launchpad.net/ubuntu/+source/epiphany-browser/+bug/116814

"Binary package hint: epiphany-browser

The "go" button acts as soon as the mouse button -be it the left or middle button- is pressed over it. This behaviour is different to other buttons' behaviours, which adopt a "pressed" appearance when the mouse button is pressed, and act when it is released.

Merely an aesthetic issue, but there it is.
...
I'm using Gutsy. The "go" button has been recently patched to accept middle-clicks, that might be related to this bug."
Comment 1 Sebastiaan Samyn 2007-05-28 19:49:53 UTC
problem is caused by the patch for this bug: http://bugzilla.gnome.org/show_bug.cgi?id=362591

The patch connected the GO button to the button-pressed-event, I think it should be connected to the button-released-event.
Comment 2 Sebastiaan Samyn 2007-05-28 20:15:04 UTC
Created attachment 88960 [details] [review]
Solution for bug 441419.

The only difference is the event type that is changed from "button-press-event" to "button-release-event". This solves this little inconsistency.
Comment 3 Christian Persch 2007-05-28 21:33:30 UTC
Somehow the patch seems to be inverted? Anyway applied with -R it's the right thing, thanks for the patch!
Comment 4 Diego Escalante Urrelo (not reading bugmail) 2007-06-05 20:21:03 UTC
Thanks Sebastiaan!, committed to trunk as revision 7072.
Should this go to 2.18 too?

2007-06-05  Diego Escalante Urrelo  <diegoe@gnome.org>

        * src/ephy-go-action.c:

        Fix go button behaviour to react to button-release-event and not
        button-press-event. Patch by Sebastiaan Samyn. Bug #441419.
Comment 5 Diego Escalante Urrelo (not reading bugmail) 2007-06-05 20:47:25 UTC
Committed to 2.18.