ICEpdf
  1. ICEpdf
  2. PDF-556

View level ToolHandlers are not getting uninstalled correctly.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 5.0.0 beta1
    • Fix Version/s: 5.0
    • Component/s: Core/Parsing, Viewer RI
    • Labels:
      None
    • Environment:
      any

      Description

      While doing some smoke testing of the release bundles I came across an issue where the view tool handler's where not correctly uninstalling when the page view tools where used. The side effect was that the mouse listeners where getting registered more then once and becoming sticky.

        Activity

        Patrick Corless created issue -
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #34250 Fri Apr 05 12:53:36 MDT 2013 patrick.corless PDF-556 fixed an issue where the view tools were not uninstalling when a new page view was selected. Also slipped in a new cursor icon for the dynamic zoom tool.
        Files Changed
        Commit graph MODIFY /icepdf/trunk/icepdf/viewer/src/org/icepdf/ri/common/SwingController.java
        Commit graph MODIFY /icepdf/trunk/icepdf/viewer/src/org/icepdf/ri/common/views/DocumentViewControllerImpl.java
        Commit graph MODIFY /icepdf/trunk/icepdf/viewer/src/org/icepdf/ri/common/MouseWheelListenerPageChanger.java
        Commit graph MODIFY /icepdf/trunk/icepdf/viewer/src/org/icepdf/ri/common/views/AbstractDocumentView.java
        Commit graph MODIFY /icepdf/trunk/icepdf/viewer/src/org/icepdf/ri/common/views/DocumentViewController.java
        Commit graph MODIFY /icepdf/trunk/icepdf/viewer/src/org/icepdf/ri/common/SwingViewBuilder.java
        Commit graph MODIFY /icepdf/trunk/icepdf/ReadMe.html
        Commit graph ADD /icepdf/trunk/icepdf/viewer/src/org/icepdf/ri/images/zoom.gif
        Hide
        Patrick Corless added a comment -

        Correct issue where the tool handler were not getting correctly uninstalled when a view changed.

        Show
        Patrick Corless added a comment - Correct issue where the tool handler were not getting correctly uninstalled when a view changed.
        Patrick Corless made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Patrick Corless made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: