Sakurai/pdf.js
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove the |el| property in PDFPageView and PDFThumbnailView

Browse Source
This commit is contained in:
Jonas Jenwald 2015-02-08 17:40:52 +01:00
parent bc7a110743
commit eed67ea8bb
3 changed files with 4 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
web/pdf_page_view.js
View File

@ -80,7 +80,6 @@ var PDFPageView = (function PDFPageViewClosure() {
div.className = 'page'; div.className = 'page';
div.style.width = Math.floor(this.viewport.width) + 'px'; div.style.width = Math.floor(this.viewport.width) + 'px';
div.style.height = Math.floor(this.viewport.height) + 'px'; div.style.height = Math.floor(this.viewport.height) + 'px';
this.el = div; // TODO replace 'el' property usage
this.div = div; this.div = div;
container.appendChild(div); container.appendChild(div);

1
web/pdf_thumbnail_view.js
View File

@ -99,7 +99,6 @@ var PDFThumbnailView = (function PDFThumbnailViewClosure() {
var div = document.createElement('div'); var div = document.createElement('div');
div.id = 'thumbnailContainer' + id; div.id = 'thumbnailContainer' + id;
div.className = 'thumbnail'; div.className = 'thumbnail';
this.el = div; // TODO: replace 'el' property usage.
this.div = div; this.div = div;
if (id === 1) { if (id === 1) {

8
web/ui_utils.js
View File

@ -194,16 +194,16 @@ function getVisibleElements(scrollEl, views, sortByVisibility) {
var currentWidth, viewWidth; var currentWidth, viewWidth;
for (var i = 0, ii = views.length; i < ii; ++i) { for (var i = 0, ii = views.length; i < ii; ++i) {
view = views[i]; view = views[i];
currentHeight = view.el.offsetTop + view.el.clientTop; currentHeight = view.div.offsetTop + view.div.clientTop;
viewHeight = view.el.clientHeight; viewHeight = view.div.clientHeight;
if ((currentHeight + viewHeight) < top) { if ((currentHeight + viewHeight) < top) {
continue; continue;
} }
if (currentHeight > bottom) { if (currentHeight > bottom) {
break; break;
} }
currentWidth = view.el.offsetLeft + view.el.clientLeft; currentWidth = view.div.offsetLeft + view.div.clientLeft;
viewWidth = view.el.clientWidth; viewWidth = view.div.clientWidth;
if ((currentWidth + viewWidth) < left || currentWidth > right) { if ((currentWidth + viewWidth) < left || currentWidth > right) {
continue; continue;
} }