[gnome-documents/gnome-3-16] preview: Simplify the PreviewNavControls setModel code
- From: Debarshi Ray <debarshir src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-documents/gnome-3-16] preview: Simplify the PreviewNavControls setModel code
- Date: Wed, 4 Mar 2015 00:13:01 +0000 (UTC)
commit dcbdf9187c5134994c6491a65dec8176691e5f9f
Author: Debarshi Ray <debarshir gnome org>
Date: Sun Mar 1 01:06:35 2015 +0100
preview: Simplify the PreviewNavControls setModel code
There is no need to mix setting the model and updating the visibility
of the controls. Once a document has finished loading we set the model
and call the show method to reveal the controls. When we leave the
preview, we call hide or destroy the controls.
This is made possible by consolidating the load-finished code in the
previous patch.
https://bugzilla.gnome.org/show_bug.cgi?id=745507
src/preview.js | 3 ---
1 files changed, 0 insertions(+), 3 deletions(-)
---
diff --git a/src/preview.js b/src/preview.js
index 2deac71..ea22671 100644
--- a/src/preview.js
+++ b/src/preview.js
@@ -736,9 +736,6 @@ const PreviewNavControls = new Lang.Class({
if (this._model)
this._pageChangedId = this._model.connect('page-changed', Lang.bind(this,
this._updateVisibility));
-
- this._updateVisibility();
- this._queueAutoHide();
},
_fadeInButton: function(widget) {
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]