ICEpdf
  1. ICEpdf
  2. PDF-546

View level text selection throwing null pointers.

          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: Viewer RI
          • Labels:
            None
          • Environment:
            any

            Description

            When starting text selecting from outside of a page will load the TextSelectionViewHandler. There was last minute refactor in beta1 where a CommonToolHandler class was introduced. This refactor was introduced a bug in TextSelectionViewHandler effectively breaking the functionality.

              Activity

              Ascending order - Click to sort in descending order
              Patrick Corless created issue -
              Hide
              Permalink
              Patrick Corless added a comment -

              Updated the TextSelectionViewHandler so that it correct accessed the parent view container instead of the null pageViewComponent.

              Show
              Patrick Corless added a comment - Updated the TextSelectionViewHandler so that it correct accessed the parent view container instead of the null pageViewComponent.
              Patrick Corless made changes -
              Field Original Value New Value
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #34171 Mon Apr 01 12:23:56 MDT 2013 patrick.corless PDF-546 fixed issue where pageViewComp was being used for a PageView level tool.
              Files Changed
              Commit graph MODIFY /icepdf/trunk/icepdf/viewer/src/org/icepdf/ri/common/tools/TextSelectionViewHandler.java
              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: