[gnome-documents] preview: use next_page/previous_page() on the view to change pages
- From: Cosimo Cecchi <cosimoc src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-documents] preview: use next_page/previous_page() on the view to change pages
- Date: Wed, 27 Mar 2013 18:48:15 +0000 (UTC)
commit 25f9491a8f5cf26a93e742893a85e5220c32049f
Author: Cosimo Cecchi <cosimoc gnome org>
Date: Wed Mar 27 14:46:29 2013 -0400
preview: use next_page/previous_page() on the view to change pages
This ensures page switching works correctly when the document is showing
in dual mode.
https://bugzilla.gnome.org/show_bug.cgi?id=694936
src/preview.js | 14 ++++++--------
1 files changed, 6 insertions(+), 8 deletions(-)
---
diff --git a/src/preview.js b/src/preview.js
index cf1918b..0bde66d 100644
--- a/src/preview.js
+++ b/src/preview.js
@@ -80,7 +80,7 @@ const PreviewView = new Lang.Class({
this._overlay.add_overlay(this._navBar.widget);
// create page nav buttons
- this._navButtons = new PreviewNavButtons(this._model, this._overlay);
+ this._navButtons = new PreviewNavButtons(this, this._overlay);
this.widget.show_all();
@@ -611,8 +611,9 @@ const _AUTO_HIDE_TIMEOUT = 2;
const PreviewNavButtons = new Lang.Class({
Name: 'PreviewNavButtons',
- _init: function(model, overlay) {
- this._model = model;
+ _init: function(previewView, overlay) {
+ this._previewView = previewView;
+ this._model = previewView.getModel();
this._overlay = overlay;
this._visible = false;
this._pageChangedId = 0;
@@ -674,14 +675,11 @@ const PreviewNavButtons = new Lang.Class({
},
_onPrevClicked: function() {
- if (this._model.page > 0)
- this._model.page--;
+ this._previewView.view.previous_page();
},
_onNextClicked: function() {
- let doc = this._model.document;
- if (doc.get_n_pages() > this._model.page + 1)
- this._model.page++;
+ this._previewView.view.next_page();
},
_autoHide: function() {
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]