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 760487 - Can't pan map after dismissing "Export as image" dialog until mouse leaves mapView
Can't pan map after dismissing "Export as image" dialog until mouse leaves ma...
Status: RESOLVED FIXED
Product: gnome-maps
Classification: Applications
Component: general
git master
Other Linux
: Normal normal
: ---
Assigned To: gnome-maps-maint
gnome-maps-maint
Depends on:
Blocks:
 
 
Reported: 2016-01-11 20:08 UTC by Hashem Nasarat
Modified: 2016-01-30 11:14 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Add MAPS_DEBUG=focus (1.19 KB, patch)
2016-01-12 08:35 UTC, Jonas Danielsson
committed Details | Review

Description Hashem Nasarat 2016-01-11 20:08:56 UTC
http://webm.land/w/GqOe/
Comment 1 Jonas Danielsson 2016-01-11 20:13:55 UTC
Thanks for filing this. I've been aware of this for a while and poked at it a bit. What happens is that the focus is on the ChamplainView. You can see this by using the arrow keys after dismissing the dialog, and notice that you can pan with them.

Not sure what the solution is, but if anyone wants to fix this, it would be nice :)
Comment 2 Jonas Danielsson 2016-01-12 08:35:34 UTC
Created attachment 318844 [details] [review]
Add MAPS_DEBUG=focus

Starting with MAPS_DEBUG=focus would keep track of the focus widget
and print messages when it changes in the form of:

   * focus widget: [object instance proxy GType:Gjs_MapView ...
Comment 3 Jonas Danielsson 2016-01-12 08:36:24 UTC
So the focus thing might be a false lead... but maybe a debug feature like the patch above is useful anyway?
Comment 4 Marcus Lundblad 2016-01-12 09:21:56 UTC
(In reply to Jonas Danielsson from comment #2)
> Created attachment 318844 [details] [review] [review]
> Add MAPS_DEBUG=focus
> 
> Starting with MAPS_DEBUG=focus would keep track of the focus widget
> and print messages when it changes in the form of:
> 
>    * focus widget: [object instance proxy GType:Gjs_MapView ...

LGTM
Comment 5 Marcus Lundblad 2016-01-12 09:23:31 UTC
And I think having a debug feature like this could be useful anyway, and it doesn't "get into the way" for normal usage, so it should be pretty unintrusive.
Comment 6 Jonas Danielsson 2016-01-30 10:55:52 UTC
Attachment 318844 [details] pushed as c8133c4 - Add MAPS_DEBUG=focus
Comment 7 Jonas Danielsson 2016-01-30 10:56:31 UTC
Premature closing