ICEpdf
  1. ICEpdf
  2. PDF-118

Fit Width view adds an unnecessary horizontal scrollbar

    Details

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

      Description

      When selecting the Fit Width view, an unnecessary horizontal scrollbar is added to the PDF viewer.

        Activity

        Hide
        Patrick Corless added a comment -

        Updated the page fit algorithms so that fit width and height work more consistently. The old algorithms did not take into consideration the layout manager padding and insets and as a result resorted to a fudge factor for calculating the fit to width or fit to height.

        The four document view types DocumentView.getDocumentSize implementation was updated respectively to calculate a normalized page sized based on a zoom of 1.0f. The DocumentViewControllerImpl could then use this normalized value from each of the four view types to calculate accurate width and hight scaling factors. In most cases no horizontal scroll bar will be present for fit to width and no vertical scroll bar for fit to height for views that are not column based.

        Show
        Patrick Corless added a comment - Updated the page fit algorithms so that fit width and height work more consistently. The old algorithms did not take into consideration the layout manager padding and insets and as a result resorted to a fudge factor for calculating the fit to width or fit to height. The four document view types DocumentView.getDocumentSize implementation was updated respectively to calculate a normalized page sized based on a zoom of 1.0f. The DocumentViewControllerImpl could then use this normalized value from each of the four view types to calculate accurate width and hight scaling factors. In most cases no horizontal scroll bar will be present for fit to width and no vertical scroll bar for fit to height for views that are not column based.

          People

          • Assignee:
            Patrick Corless
            Reporter:
            Tyler Johnson
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: