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

Unified Diff: chrome/browser/resources/pdf/pdf.js

Issue 2283803003: PDF: Add shortcut to toggle fit-to-{page,width}. (Closed)
Patch Set: Extend testZoomToolbarToggle 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
Index: chrome/browser/resources/pdf/pdf.js
diff --git a/chrome/browser/resources/pdf/pdf.js b/chrome/browser/resources/pdf/pdf.js
index c7b172078d7778472af64c4fe851a35bca15b73e..c61f986b44e22abc4ca2b9d26cbcc638e6075954 100644
--- a/chrome/browser/resources/pdf/pdf.js
+++ b/chrome/browser/resources/pdf/pdf.js
@@ -377,7 +377,7 @@ PDFViewer.prototype = {
this.viewport.position = position;
}
return;
- case 65: // a key.
+ case 65: // 'a' key.
if (e.ctrlKey || e.metaKey) {
this.plugin_.postMessage({
type: 'selectAll'
@@ -386,17 +386,21 @@ PDFViewer.prototype = {
e.preventDefault();
}
return;
- case 71: // g key.
+ case 71: // 'g' key.
if (this.toolbar_ && (e.ctrlKey || e.metaKey) && e.altKey) {
this.toolbarManager_.showToolbars();
this.toolbar_.selectPageNumber();
}
return;
- case 219: // left bracket.
+ case 219: // Left bracket key.
if (e.ctrlKey)
this.rotateCounterClockwise_();
return;
- case 221: // right bracket.
+ case 220: // Backslash key.
+ if (e.ctrlKey)
+ this.zoomToolbar_.fitToggleFromHotKey();
+ return;
+ case 221: // Right bracket key.
if (e.ctrlKey)
this.rotateClockwise_();
return;
@@ -442,6 +446,10 @@ PDFViewer.prototype = {
});
},
+ /**
+ * @private
+ * Set zoom to "fit to page".
+ */
fitToPage_: function() {
this.viewport_.fitToPage();
this.toolbarManager_.forceHideTopToolbar();

Powered by Google App Engine
This is Rietveld 408576698