[Pkg-javascript-commits] [pdf.js] 39/106: Eliminate transition effects from thumbnail loader

David Prévot taffit at moszumanska.debian.org
Sat Jun 20 21:33:49 UTC 2015


This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to branch master
in repository pdf.js.

commit a22319b98a0f108eca5c878b59b19c9fa0076076
Author: Mike Corbin <mcorb at appshuttle.com>
Date:   Sat May 16 15:03:00 2015 +0100

    Eliminate transition effects from thumbnail loader
    
    During A/B testing we found a significant preference (increased engagement
    time, reduced downloads of original PDF for local viewing) for the version that
    doesn't animate loading thumbnails.
    
    Profiling shows that the transition effects hit at a pessimal time when the
    compositor is busy rendering the main PDF page view, and rendering thumbnail
    previews, causing the main scroll view to stutter. With the transition removed
    scrolling is perceptibly smoother in Chrome and Firefox. We also hypothesize
    that the transition effects added to a perception of slowness but presumably
    the aforementioned stutter is the primary issue here.
---
 web/viewer.css | 2 --
 1 file changed, 2 deletions(-)

diff --git a/web/viewer.css b/web/viewer.css
index 3dfae37..deb9173 100644
--- a/web/viewer.css
+++ b/web/viewer.css
@@ -1127,7 +1127,6 @@ html[dir='rtl'] .verticalToolbarSeparator {
 }
 
 .thumbnailImage {
-  transition-duration: 150ms;
   border: 1px solid transparent;
   box-shadow: 0 0 0 1px rgba(0, 0, 0, 0.5), 0 2px 8px rgba(0, 0, 0, 0.3);
   opacity: 0.8;
@@ -1139,7 +1138,6 @@ html[dir='rtl'] .verticalToolbarSeparator {
 .thumbnailSelectionRing {
   border-radius: 2px;
   padding: 7px;
-  transition-duration: 150ms;
 }
 
 a:focus > .thumbnail > .thumbnailSelectionRing > .thumbnailImage,

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-javascript/pdf.js.git



More information about the Pkg-javascript-commits mailing list