diff options
| author | Jonas Danielsson <jonas@threetimestwo.org> | 2016-01-30 12:04:16 +0100 |
|---|---|---|
| committer | Jonas Danielsson <jonas@threetimestwo.org> | 2016-01-30 12:12:12 +0100 |
| commit | f654384383efac5de127d2b65a54e32bde374e0b (patch) | |
| tree | 6c9c58a45f0c4becfba2e27acfedc683ead5bed8 /src | |
| parent | c8133c4b2808b3888754b9dc8754ad60196f5da4 (diff) | |
| download | gnome-maps-f654384383efac5de127d2b65a54e32bde374e0b.tar.gz | |
Move away from using GtkDialog::run
Instead we can connect to the response signal and destroy
the dialog in there. This seems to fix a buglet where
we could not pan after dismissing dialogs.
https://bugzilla.gnome.org/show_bug.cgi?id=760487
Diffstat (limited to 'src')
| -rw-r--r-- | src/contextMenu.js | 6 | ||||
| -rw-r--r-- | src/exportViewDialog.js | 6 | ||||
| -rw-r--r-- | src/mapBubble.js | 6 |
3 files changed, 12 insertions, 6 deletions
diff --git a/src/contextMenu.js b/src/contextMenu.js index 39633b16..87f08329 100644 --- a/src/contextMenu.js +++ b/src/contextMenu.js @@ -106,8 +106,10 @@ const ContextMenu = new Lang.Class({ mapView: this._mapView }); - dialog.run(); - dialog.destroy(); + dialog.connect('response', function() { + dialog.destroy(); + }); + dialog.show_all(); }, _onExportActivated: function() { diff --git a/src/exportViewDialog.js b/src/exportViewDialog.js index 15e1e543..251b568b 100644 --- a/src/exportViewDialog.js +++ b/src/exportViewDialog.js @@ -171,8 +171,10 @@ const ExportViewDialog = new Lang.Class({ secondary_text: details }); - dialog.run(); - dialog.destroy(); + dialog.connect('response', function() { + dialog.destroy(); + }); + dialog.show_all(); } }, diff --git a/src/mapBubble.js b/src/mapBubble.js index 5d442157..74f8b598 100644 --- a/src/mapBubble.js +++ b/src/mapBubble.js @@ -142,8 +142,10 @@ const MapBubble = new Lang.Class({ button.visible = true; button.connect('clicked', function() { - dialog.run(); - dialog.hide(); + dialog.connect('response', function() { + dialog.hide(); + }); + dialog.show_all(); }); }, |