ICEpdf
  1. ICEpdf
  2. PDF-62

Scroll page to top when setting Fit to Width

          Details

          • Type: Improvement Improvement
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 3.1
          • Fix Version/s: 4.0 - Beta, 4.0
          • Component/s: Viewer RI
          • Labels:
            None
          • Environment:
            -

            Description

            When setting fit to width, the page zooms towards the center rather than
            positioning itself at the top. The customer would like us to consider this fix
            for our next release. Here is what he has done:

            I got this working by modifying the setFitMode(int) method as follows:

             At the bottom of the method I replaced the call to setZoom(newZoom, null, true)
            with the following logic:

             // set appropriate zoom level
             // If we're scrolled all the way to the top, center to top of document when zoom.
             if (getVerticalScrollBar().getValue() == 0)
             {
               setZoom(newZoom, new Point(0, 0), true);
             }
             else
              {
               setZoom(newZoom, null, true);
             }

              Activity

              Repository Revision Date User Message
              ICEsoft Public SVN Repository #19892 Tue Dec 08 15:43:31 MST 2009 carlo.guglielmin PDF-62 - Changed Fit to Width behaviour so that if the user is scrolled to the top of the scrollbar, we maintain that state when zooming in for the fit. Otherwise (and this was the previous behaviour) the zoom tends to just go towards the center of the page. Patch was from community member Ken Walker.
              Files Changed
              Commit graph MODIFY /icepdf/trunk/icepdf/viewer/src/org/icepdf/ri/common/views/DocumentViewControllerImpl.java

                People

                • Assignee:
                  Carlo Guglielmin
                  Reporter:
                  Tyler Johnson
                • Votes:
                  1 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: