mirrors/pdf.js
1
0
Fork 0
mirror of https://github.com/mozilla/pdf.js.git synced 2025-04-21 15:48:06 +02:00
Code Activity

Change the GoToPage Named Action to select the contents of the pageNumber input, rather than just focusing the element

Browse source 
When clicking on the `pageNumber` input, or when using the keyboard shortcut <kbd>Ctrl</kbd>+<kbd>Alt</kbd>+<kbd>G</kbd>, the element isn't just focused but its contents is actually *selected* so that the user doesn't need to clear the input before entering a new `pageNumber`.
However, the `GoToPage` named action is only using `focus`, so let's change this to be consistent.

The following document can be used for testing: http://www2.informatik.uni-freiburg.de/~frank/ENG/beamer/example/Beamer-class-example1.pdf#zoom=page-fit

[screeenshot]
This commit is contained in:
Jonas Jenwald 2016-09-01 13:23:18 +02:00
parent 23b13d368e
commit 0db76dc58c
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
web/app.js
View file

@ -1707,7 +1707,7 @@ function webViewerNamedAction(e) {
var action = e.action;
switch (action) {
case 'GoToPage':
PDFViewerApplication.appConfig.toolbar.pageNumber.focus();
PDFViewerApplication.appConfig.toolbar.pageNumber.select();
break;
case 'Find':