Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2674)

Unified Diff: chrome/browser/resources/pdf/elements/viewer-zoom-toolbar/viewer-zoom-toolbar.js

Issue 2283803003: PDF: Add shortcut to toggle fit-to-{page,width}. (Closed)
Patch Set: sync button state Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/resources/pdf/pdf.js » ('j') | chrome/browser/resources/pdf/pdf.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/pdf/elements/viewer-zoom-toolbar/viewer-zoom-toolbar.js
diff --git a/chrome/browser/resources/pdf/elements/viewer-zoom-toolbar/viewer-zoom-toolbar.js b/chrome/browser/resources/pdf/elements/viewer-zoom-toolbar/viewer-zoom-toolbar.js
index 060178fcc6fe214d82b0b5d3ffdf5957b50346e7..71a72d6d3b9648aac8b91b499d023839f7e4c198 100644
--- a/chrome/browser/resources/pdf/elements/viewer-zoom-toolbar/viewer-zoom-toolbar.js
+++ b/chrome/browser/resources/pdf/elements/viewer-zoom-toolbar/viewer-zoom-toolbar.js
@@ -38,6 +38,21 @@
this.$['zoom-out-button'].tooltips = [this.strings.tooltipZoomOut];
},
+ /**
+ * Toggle the state of the fit-button when something external, e.g. the
+ * fit-button equivalent shortcut key, toggles the zoom state.
+ */
+ toggleFitButtonUIState: function() {
+ var button = this.$['fit-button'];
+ if (button.activeIndex == FIT_TO_WIDTH)
+ button.activeIndex = FIT_TO_PAGE;
+ else
+ button.activeIndex = FIT_TO_WIDTH;
+ },
+
+ /**
+ * Handle clicks of the fit-button.
+ */
fitToggle: function() {
if (this.$['fit-button'].activeIndex == FIT_TO_WIDTH)
this.fire('fit-to-width');
@@ -45,10 +60,16 @@
this.fire('fit-to-page');
},
+ /**
+ * Handle clicks of the zoom-in-button.
+ */
zoomIn: function() {
this.fire('zoom-in');
},
+ /**
+ * Handle clicks of the zoom-out-button.
+ */
zoomOut: function() {
this.fire('zoom-out');
},
« no previous file with comments | « no previous file | chrome/browser/resources/pdf/pdf.js » ('j') | chrome/browser/resources/pdf/pdf.js » ('J')

Powered by Google App Engine
This is Rietveld 408576698