From bf86d1bec855bfab88939979792914945f4e62dd Mon Sep 17 00:00:00 2001 From: Vivien Nicolas <21@vingtetun.org> Date: Fri, 29 Jul 2011 18:17:04 +0200 Subject: [PATCH 1/8] Use Tango Public Domain icons and add some small tweaks to viewer.js --- web/images/document-print.svg | 532 ++++++++++++++++++++++++++++++++++ web/images/go-down.svg | 35 +-- web/images/go-up.svg | 35 +-- web/viewer.html | 7 + web/viewer.js | 58 ++-- 5 files changed, 611 insertions(+), 56 deletions(-) create mode 100644 web/images/document-print.svg diff --git a/web/images/document-print.svg b/web/images/document-print.svg new file mode 100644 index 000000000..0b8837ba1 --- /dev/null +++ b/web/images/document-print.svg @@ -0,0 +1,532 @@ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + image/svg+xml + + Print Document + + + Jakub Steiner + + + + http://jimmac.musichall.cz + + + document + lpr + print + local + laser + bubblejet + inkjet + print + output + cups + lpd + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + diff --git a/web/images/go-down.svg b/web/images/go-down.svg index 18dadc498..95b82afd1 100644 --- a/web/images/go-down.svg +++ b/web/images/go-down.svg @@ -2,16 +2,16 @@ + http://jimmac.musichall.cz + rdf:resource="http://creativecommons.org/licenses/publicdomain/" /> Go Down @@ -146,19 +153,13 @@ + rdf:about="http://creativecommons.org/licenses/publicdomain/"> + rdf:resource="http://creativecommons.org/ns#Reproduction" /> - - + rdf:resource="http://creativecommons.org/ns#Distribution" /> - + rdf:resource="http://creativecommons.org/ns#DerivativeWorks" /> diff --git a/web/images/go-up.svg b/web/images/go-up.svg index 0e3d01d17..54263df3e 100644 --- a/web/images/go-up.svg +++ b/web/images/go-up.svg @@ -2,16 +2,16 @@ + http://jimmac.musichall.cz + rdf:resource="http://creativecommons.org/licenses/publicdomain/" /> Go Up @@ -146,19 +153,13 @@ + rdf:about="http://creativecommons.org/licenses/publicdomain/"> + rdf:resource="http://creativecommons.org/ns#Reproduction" /> - - + rdf:resource="http://creativecommons.org/ns#Distribution" /> - + rdf:resource="http://creativecommons.org/ns#DerivativeWorks" /> diff --git a/web/viewer.html b/web/viewer.html index 285dadb01..e740c2bc0 100644 --- a/web/viewer.html +++ b/web/viewer.html @@ -44,6 +44,13 @@
+ + +
+
diff --git a/web/viewer.js b/web/viewer.js index 328388396..eae5b383e 100644 --- a/web/viewer.js +++ b/web/viewer.js @@ -5,7 +5,6 @@ var kDefaultURL = 'compressed.tracemonkey-pldi-09.pdf'; var kDefaultScale = 150; - var kCacheSize = 20; var Cache = function(size) { @@ -24,41 +23,40 @@ var PDFView = { thumbnails: [], set scale(val) { - var options = document.getElementById('scaleSelect').options; - for (var i = 0; i < options.length; i++) { - var option = options[i]; - option.selected = (option.value == val); - } - var pages = this.pages; var cssUnits = 96.0 / 72.0; for (var i = 0; i < pages.length; i++) pages[i].update(val / 100 * cssUnits); // Jump the scroll position to the correct page. - this.page = this.page; + document.location.hash = this.page; + + var event = document.createEvent("UIEvents"); + event.initUIEvent("scalechange", false, false, window, val); + window.dispatchEvent(event); }, set page(val) { var pages = this.pages; - var input = document.getElementById("pageNumber"); - if (val <= 0 || val > pages.length) { - input.value = this.page; - return; + if (val <= 0 || val == this.page || val > pages.length) { + // TODO If the hash if set to a dumb value, like #123456, the input field + // of the UI will be set to it even if no page is changed because its out + // of bound. + val = this.page || 1; + } else { + // Draw the page before jumping to it in order to avoid seeing the + // possible gap between pages if the page has never been draw before. + pages[val - 1].draw(); + document.location.hash = val; } - document.location.hash = val; - document.getElementById("previous").disabled = (val == 1); - document.getElementById("next").disabled = (val == pages.length); - if (input.value == val) - return; - - input.value = val; - pages[val - 1].draw(); + var event = document.createEvent("UIEvents"); + event.initUIEvent("pagechange", false, false, window, val); + window.dispatchEvent(event); }, get page() { - return parseInt(document.location.hash.substring(1)) || 1; + return parseInt(document.location.hash.substring(1)); }, open: function(url, scale) { @@ -253,7 +251,7 @@ window.addEventListener('pdfloaded', function(evt) { PDFView.load(evt.detail); }, true); -window.addEventListener('scroll', function(evt) { +window.addEventListener('scroll', function onscroll(evt) { var visiblePages = PDFView.getVisiblePages(); for (var i = 0; i < visiblePages.length; i++) { var page = visiblePages[i]; @@ -316,3 +314,19 @@ window.addEventListener("transitionend", function(evt) { }, 500); }, true); + +window.addEventListener("scalechange", function(evt) { + var options = document.getElementById('scaleSelect').options; + for (var i = 0; i < options.length; i++) { + var option = options[i]; + option.selected = (option.value == evt.detail); + } +}, true); + +window.addEventListener("pagechange", function(evt) { + var page = evt.detail; + document.location.hash = page; + document.getElementById("pageNumber").value = page; + document.getElementById("previous").disabled = (page == 1); + document.getElementById("next").disabled = (page == PDFView.pages.length); +}, true); From 3aa6e38f01c37851d8815d5d5710532699a2a038 Mon Sep 17 00:00:00 2001 From: notmasteryet Date: Sun, 21 Aug 2011 21:05:10 -0500 Subject: [PATCH 2/8] Switching to the document outline view if the document structure is present --- pdf.js | 39 +++-- web/images/nav-outline.svg | 202 ++++++++++++++++++++++++++ web/images/nav-thumbs.svg | 283 +++++++++++++++++++++++++++++++++++++ web/viewer.css | 76 +++++++++- web/viewer.html | 11 ++ web/viewer.js | 63 +++++++++ 6 files changed, 657 insertions(+), 17 deletions(-) create mode 100644 web/images/nav-outline.svg create mode 100644 web/images/nav-thumbs.svg diff --git a/pdf.js b/pdf.js index 0ff3381f5..0d44822dd 100644 --- a/pdf.js +++ b/pdf.js @@ -3395,6 +3395,9 @@ var Page = (function() { TODO('other link types'); break; } + } else if (annotation.has('Dest')) { + // simple destination link + link.dest = annotation.get('Dest').name; } links.push(link); } @@ -3449,11 +3452,12 @@ var Catalog = (function() { var outlineDict = this.xref.fetch(i.obj); if (!outlineDict.has('Title')) error('Invalid outline item'); - var dest = outlineDict.get('Dest'); - if (!dest && outlineDict.get('A')) { - var a = this.xref.fetchIfRef(outlineDict.get('A')); - dest = a.get('D'); - } + var dest = outlineDict.get('A'); + if (dest) + dest = this.xref.fetchIfRef(dest).get('D'); + else if (outlineDict.has('Dest')) + dest = outlineDict.get('Dest').name; + var outlineItem = { dest: dest, title: convertIfUnicode(outlineDict.get('Title')), @@ -3477,7 +3481,8 @@ var Catalog = (function() { } } } - return shadow(this, 'documentOutline', root); + obj = root.items.length > 0 ? root.items : null; + return shadow(this, 'documentOutline', obj); }, get numPages() { var obj = this.toplevelPagesDict.get('Count'); @@ -3511,15 +3516,25 @@ var Catalog = (function() { }, get destinations() { var xref = this.xref; + var dests = {}, nameTreeRef, nameDictionaryRef; var obj = this.catDict.get('Names'); - obj = obj ? xref.fetch(obj) : this.catDict; - obj = obj.get('Dests'); - var dests = {}; - if (obj) { + if (obj) + nameTreeRef = xref.fetch(obj).get('Dests'); + else if(this.catDict.has('Dests')) + nameDictionaryRef = this.catDict.get('Dests'); + + if (nameDictionaryRef) { + // reding simple destination dictionary + obj = xref.fetch(nameDictionaryRef); + obj.forEach(function(key, value) { + dests[key] = xref.fetch(value).get('D'); + }); + } + if (nameTreeRef) { // reading name tree var processed = new RefSet(); - processed.put(obj); - var queue = [obj]; + processed.put(nameTreeRef); + var queue = [nameTreeRef]; while (queue.length > 0) { var i, n; obj = xref.fetch(queue.shift()); diff --git a/web/images/nav-outline.svg b/web/images/nav-outline.svg new file mode 100644 index 000000000..4d4323ce3 --- /dev/null +++ b/web/images/nav-outline.svg @@ -0,0 +1,202 @@ + + + + + + + + + + + + + + image/svg+xml + + + + + + + + + + + + + + + + + + + + + + + + + + + diff --git a/web/images/nav-thumbs.svg b/web/images/nav-thumbs.svg new file mode 100644 index 000000000..8737b8cb6 --- /dev/null +++ b/web/images/nav-thumbs.svg @@ -0,0 +1,283 @@ + + + + + + + + + + + + + + + + + image/svg+xml + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + diff --git a/web/viewer.css b/web/viewer.css index 54f648adb..e18d9681e 100644 --- a/web/viewer.css +++ b/web/viewer.css @@ -60,10 +60,10 @@ span#info { /* === Sidebar === */ #sidebar { position: fixed; - width: 200px; + width: 350px; top: 62px; bottom: 18px; - left: -140px; + left: -290px; transition: left 0.25s ease-in-out 1s; -moz-transition: left 0.25s ease-in-out 1s; -webkit-transition: left 0.25s ease-in-out 1s; @@ -78,7 +78,7 @@ span#info { #sidebarBox { background-color: rgba(0, 0, 0, 0.7); - width: 150px; + width: 300px; height: 100%; border-top-right-radius: 8px; border-bottom-right-radius: 8px; @@ -98,14 +98,73 @@ span#info { top: 10px; bottom: 10px; left: 10px; - width: 130px; + width: 280px; } .thumbnail { width: 104px; height: 134px; background-color: white; - margin: 5px; + margin-top: 5px; + margin-bottom: 5px; + margin-left:auto; + margin-right:auto; +} + +#outlineScrollView { + position: absolute; + background-color: #fff; + overflow: auto; + top: 10px; + bottom: 10px; + left: 10px; + width: 280px; +} + +#outlineView { + padding-top: 4px; + padding-bottom: 100px; + padding-left: 6px; + padding-right: 6px; + font-size: smaller; +} + +.outlineItem > .outlineItems { + margin-left: 20px; +} + +.outlineItem > a { + text-decoration: none; + color: black; +} + +.outlineItem > a:hover { + background: #ff0; + box-shadow: 0px 2px 10px #ff0; +} + +#sidebarControls { + position:absolute; + width: 120px; + height: 32px; + left: 15px; + bottom: 35px; +} + +#sidebarControls > button { + box-shadow: 0px 4px 10px #000; + -moz-box-shadow: 0px 4px 10px #000; + -webkit-box-shadow: 0px 4px 10px #000; +} + +#sidebarControls > button[disabled] > img { + opacity: 0.5; +} + +#sidebarControls > button[data-selected] { + box-shadow: 0px 4px 10px #ff0; + -moz-box-shadow: 0px 4px 10px #ff0; + -webkit-box-shadow: 0px 4px 10px #ff0; } /* === Content view === */ @@ -128,3 +187,10 @@ canvas { padding: 8px 0px; } +#sidebarView canvas:hover { + background: #ff0; + box-shadow: 0px 2px 10px #ff0; + -moz-box-shadow: 0px 2px 10px #ff0; + -webkit-box-shadow: 0px 2px 10px #ff0; +} + diff --git a/web/viewer.html b/web/viewer.html index 285dadb01..6232ce4b3 100644 --- a/web/viewer.html +++ b/web/viewer.html @@ -56,6 +56,17 @@
+ +
+ + +
diff --git a/web/viewer.js b/web/viewer.js index 8fe011b29..95a32128c 100644 --- a/web/viewer.js +++ b/web/viewer.js @@ -128,6 +128,33 @@ var PDFView = { this.page = parseInt(document.location.hash.substring(1)) || 1; this.pagesRefMap = pagesRefMap; this.destinations = pdf.catalog.destinations; + if (pdf.catalog.documentOutline) { + this.outline = new DocumentOutlineView(pdf.catalog.documentOutline); + var outlineSwitchButton = document.getElementById('outlineSwitch'); + outlineSwitchButton.removeAttribute('disabled'); + this.switchSidebarView('outline'); + } + }, + + switchSidebarView: function(view) { + var thumbsScrollView = document.getElementById('sidebarScrollView'); + var outlineScrollView = document.getElementById('outlineScrollView'); + var thumbsSwitchButton = document.getElementById('thumbsSwitch'); + var outlineSwitchButton = document.getElementById('outlineSwitch'); + switch(view) { + case 'thumbs': + thumbsScrollView.style.display = 'block'; + outlineScrollView.style.display = 'none'; + thumbsSwitchButton.setAttribute('data-selected', true); + outlineSwitchButton.removeAttribute('data-selected'); + break; + case 'outline': + thumbsScrollView.style.display = 'none'; + outlineScrollView.style.display = 'block'; + thumbsSwitchButton.removeAttribute('data-selected'); + outlineSwitchButton.setAttribute('data-selected', true); + break; + } }, getVisiblePages: function() { @@ -289,6 +316,42 @@ var ThumbnailView = function(container, page) { }; }; +var DocumentOutlineView = function(outline) { + var outlineView = document.getElementById('outlineView'); + + function bindItemLink(domObj, item) { + domObj.href = ''; + domObj.onclick = function(e) { + PDFView.navigateTo(item.dest); + return false; + }; + } + + var queue = [{parent: outlineView, items: outline}]; + while (queue.length > 0) { + var levelData = queue.shift(); + var i, n = levelData.items.length; + for (i = 0; i < n; i++) { + var item = levelData.items[i]; + var div = document.createElement('div'); + div.className = 'outlineItem'; + var a = document.createElement('a'); + bindItemLink(a, item); + a.textContent = item.title; + div.appendChild(a); + + if (item.items.length > 0) { + var itemsDiv = document.createElement('div'); + itemsDiv.className = 'outlineItems'; + div.appendChild(itemsDiv); + queue.push({parent: itemsDiv, items: item.items}); + } + + levelData.parent.appendChild(div); + } + } +}; + window.addEventListener('load', function(evt) { var params = document.location.search.substring(1).split('&'); for (var i = 0; i < params.length; i++) { From 94eac2466e3002e78e512169a4a51197c14d0c4c Mon Sep 17 00:00:00 2001 From: notmasteryet Date: Sun, 21 Aug 2011 21:16:17 -0500 Subject: [PATCH 3/8] Document outline item link (for an-open-web) --- pdf.js | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/pdf.js b/pdf.js index 0d44822dd..185db1f5d 100644 --- a/pdf.js +++ b/pdf.js @@ -3455,8 +3455,11 @@ var Catalog = (function() { var dest = outlineDict.get('A'); if (dest) dest = this.xref.fetchIfRef(dest).get('D'); - else if (outlineDict.has('Dest')) - dest = outlineDict.get('Dest').name; + else if (outlineDict.has('Dest')) { + dest = outlineDict.get('Dest'); + if (IsName(dest)) + dest = dest.name; + } var outlineItem = { dest: dest, From d664652b866cb544b7ef95443d07b6cd261c5732 Mon Sep 17 00:00:00 2001 From: notmasteryet Date: Mon, 22 Aug 2011 18:55:49 -0500 Subject: [PATCH 4/8] Lifting up this.xref; buttons tooltips --- pdf.js | 7 ++++--- web/viewer.html | 8 ++++---- 2 files changed, 8 insertions(+), 7 deletions(-) diff --git a/pdf.js b/pdf.js index 185db1f5d..c43a9d61c 100644 --- a/pdf.js +++ b/pdf.js @@ -3438,9 +3438,10 @@ var Catalog = (function() { return str; } var obj = this.catDict.get('Outlines'); + var xref = this.xref; var root = { items: [] }; if (IsRef(obj)) { - obj = this.xref.fetch(obj).get('First'); + obj = xref.fetch(obj).get('First'); var processed = new RefSet(); if (IsRef(obj)) { var queue = [{obj: obj, parent: root}]; @@ -3449,12 +3450,12 @@ var Catalog = (function() { processed.put(obj); while (queue.length > 0) { var i = queue.shift(); - var outlineDict = this.xref.fetch(i.obj); + var outlineDict = xref.fetch(i.obj); if (!outlineDict.has('Title')) error('Invalid outline item'); var dest = outlineDict.get('A'); if (dest) - dest = this.xref.fetchIfRef(dest).get('D'); + dest = xref.fetchIfRef(dest).get('D'); else if (outlineDict.has('Dest')) { dest = outlineDict.get('Dest'); if (IsName(dest)) diff --git a/web/viewer.html b/web/viewer.html index 6232ce4b3..2766412e0 100644 --- a/web/viewer.html +++ b/web/viewer.html @@ -60,11 +60,11 @@
- -
From 406c2b2f2b5dfeda4c5ab1132965e9ad7844e05c Mon Sep 17 00:00:00 2001 From: Vivien Nicolas <21@vingtetun.org> Date: Tue, 23 Aug 2011 16:49:25 +0200 Subject: [PATCH 5/8] Fix a small typo error with the kRasterizerMin usage --- pdf.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pdf.js b/pdf.js index 469eec578..be585046f 100644 --- a/pdf.js +++ b/pdf.js @@ -4727,7 +4727,7 @@ var CanvasGraphics = (function() { var scaleFactorX = 1, scaleFactorY = 1; var font = this.current.font; if (font) { - if (this.current.fontSize < kRasterizerMin) { + if (this.current.fontSize <= kRasterizerMin) { scaleFactorX = scaleFactorY = kScalePrecision; ctx.scale(1 / scaleFactorX, 1 / scaleFactorY); } From 35cc5f9965838474c47f94d787a85915e2bead67 Mon Sep 17 00:00:00 2001 From: mohansun Date: Tue, 23 Aug 2011 21:46:11 -0300 Subject: [PATCH 6/8] fix for the issue:https://github.com/andreasgal/pdf.js/issues/360 --- web/viewer.js | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/web/viewer.js b/web/viewer.js index f0ecd4667..698b8974d 100644 --- a/web/viewer.js +++ b/web/viewer.js @@ -27,9 +27,12 @@ var PDFView = { var cssUnits = 96.0 / 72.0; for (var i = 0; i < pages.length; i++) pages[i].update(val / 100 * cssUnits); - - // Jump the scroll position to the correct page. - document.location.hash = this.page; + + if(document.location.hash == '#' + this.page) + this.pages[this.page-1].draw(); + else + // Jump the scroll position to the correct page. + document.location.hash = this.page; var event = document.createEvent("UIEvents"); event.initUIEvent("scalechange", false, false, window, val); From 1babe72c382722c1c2081129d7e4e9ed9410984e Mon Sep 17 00:00:00 2001 From: mohansun Date: Tue, 23 Aug 2011 22:00:12 -0300 Subject: [PATCH 7/8] Fix for few html tag errors (as reported by Safari 5.0.6) --- web/viewer.html | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/web/viewer.html b/web/viewer.html index e740c2bc0..0e779c60c 100644 --- a/web/viewer.html +++ b/web/viewer.html @@ -2,7 +2,7 @@ Simple pdf.js page viewer - + @@ -26,7 +26,7 @@
- + / -- @@ -38,7 +38,7 @@ - + @@ -51,7 +51,7 @@
- +
From 36d9462016a83d3c968e99269f699a6bbe3e2e0e Mon Sep 17 00:00:00 2001 From: notmasteryet Date: Tue, 23 Aug 2011 16:58:41 -0500 Subject: [PATCH 8/8] comment fix --- pdf.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pdf.js b/pdf.js index c43a9d61c..962f76398 100644 --- a/pdf.js +++ b/pdf.js @@ -3528,7 +3528,7 @@ var Catalog = (function() { nameDictionaryRef = this.catDict.get('Dests'); if (nameDictionaryRef) { - // reding simple destination dictionary + // reading simple destination dictionary obj = xref.fetch(nameDictionaryRef); obj.forEach(function(key, value) { dests[key] = xref.fetch(value).get('D');