Merge pull request #1131 from notmasteryet/tree-61
Fixing initial scale when named destination is specified
This commit is contained in:
		
						commit
						5841383d97
					
				| @ -6,6 +6,7 @@ | |||||||
| var kDefaultURL = 'compressed.tracemonkey-pldi-09.pdf'; | var kDefaultURL = 'compressed.tracemonkey-pldi-09.pdf'; | ||||||
| var kDefaultScale = 'auto'; | var kDefaultScale = 'auto'; | ||||||
| var kDefaultScaleDelta = 1.1; | var kDefaultScaleDelta = 1.1; | ||||||
|  | var kUnknownScale = 0; | ||||||
| var kCacheSize = 20; | var kCacheSize = 20; | ||||||
| var kCssUnits = 96.0 / 72.0; | var kCssUnits = 96.0 / 72.0; | ||||||
| var kScrollbarPadding = 40; | var kScrollbarPadding = 40; | ||||||
| @ -148,7 +149,7 @@ var currentPageNumber = 1; | |||||||
| var PDFView = { | var PDFView = { | ||||||
|   pages: [], |   pages: [], | ||||||
|   thumbnails: [], |   thumbnails: [], | ||||||
|   currentScale: 0, |   currentScale: kUnknownScale, | ||||||
|   currentScaleValue: null, |   currentScaleValue: null, | ||||||
|   initialBookmark: document.location.hash.substring(1), |   initialBookmark: document.location.hash.substring(1), | ||||||
| 
 | 
 | ||||||
| @ -452,10 +453,16 @@ var PDFView = { | |||||||
|     } |     } | ||||||
|     else if (storedHash) |     else if (storedHash) | ||||||
|       this.setHash(storedHash); |       this.setHash(storedHash); | ||||||
|     else { |     else if (scale) { | ||||||
|       this.parseScale(scale || kDefaultScale, true); |       this.parseScale(scale, true); | ||||||
|       this.page = 1; |       this.page = 1; | ||||||
|     } |     } | ||||||
|  | 
 | ||||||
|  |     if (PDFView.currentScale === kUnknownScale) { | ||||||
|  |       // Scale was not initialized: invalid bookmark or scale was not specified.
 | ||||||
|  |       // Setting the default one.
 | ||||||
|  |       this.parseScale(kDefaultScale, true); | ||||||
|  |     } | ||||||
|   }, |   }, | ||||||
| 
 | 
 | ||||||
|   setHash: function pdfViewSetHash(hash) { |   setHash: function pdfViewSetHash(hash) { | ||||||
| @ -742,6 +749,8 @@ var PageView = function pageView(container, content, id, pageWidth, pageHeight, | |||||||
| 
 | 
 | ||||||
|       if (scale && scale !== PDFView.currentScale) |       if (scale && scale !== PDFView.currentScale) | ||||||
|         PDFView.parseScale(scale, true); |         PDFView.parseScale(scale, true); | ||||||
|  |       else if (PDFView.currentScale === kUnknownScale) | ||||||
|  |         PDFView.parseScale(kDefaultScale, true); | ||||||
| 
 | 
 | ||||||
|       setTimeout(function pageViewScrollIntoViewRelayout() { |       setTimeout(function pageViewScrollIntoViewRelayout() { | ||||||
|         // letting page to re-layout before scrolling
 |         // letting page to re-layout before scrolling
 | ||||||
|  | |||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user