ICEpdf
  1. ICEpdf
  2. PDF-1103

Text Selection hybrid approach

          Details

          • Type: New Feature New Feature
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: None
          • Fix Version/s: 6.3
          • Component/s: Viewer RI
          • Labels:
            None
          • Environment:
            Any

            Description

            PDF-990 captured a major rework of the the text selection algorithm from a text box inclusion mechanism to mouse based start/end detection.

            The text box solution work well but had a few corner case issue that didn't allow text on two lines to be easily selected but it work very well for multiple page selections. The new approach in PDF-990 worked a lot better for selecting text on a line by line bases but was susceptible to missing lines if a user moves very quickly from one page to the next.

            This bug will capture the work needed to bring these two approaches together to get a solution that has no limitation.

              Activity

              Ascending order - Click to sort in descending order
              Patrick Corless created issue -
              Patrick Corless made changes -
              Field Original Value New Value
              Fix Version/s 6.3 [ 13093 ]
              Hide
              Permalink
              Patrick Corless added a comment -

              A community memory has posted an test file that will be very helpful while reworking our text extraction sorting algorithms. The test file can be found here, http://www.icesoft.org/JForum/posts/list/23297.page .

              Show
              Patrick Corless added a comment - A community memory has posted an test file that will be very helpful while reworking our text extraction sorting algorithms. The test file can be found here, http://www.icesoft.org/JForum/posts/list/23297.page .
              Arran Mccullough made changes -
              Support Case References Support Case #14032 - https://icesoft.my.salesforce.com/5007000001hjGrM
              Support Case #14057 - https://icesoft.my.salesforce.com/5007000001iLXD5
              Hide
              Permalink
              Patrick Corless added a comment -

              There has also been a request to extend the top and bottom margin definitions to include left and right. We'll need to do quite a bit of refactoring to tie this all together.

              Show
              Patrick Corless added a comment - There has also been a request to extend the top and bottom margin definitions to include left and right. We'll need to do quite a bit of refactoring to tie this all together.
              Hide
              Permalink
              Patrick Corless added a comment -

              I've done quite a bit of work on text selection as a whole. The main area of concern was with regards to missing selected text when mousing quickly. The new rules address the missing lines and in general make selection feel a little smoother.

              There are still a few corners cases where selection might seem unpredictable. This is mainly related to text ordering on a page. We attempt to sort text for a given page but there are cases such as columns were a full sort would actually break the ordering the selected text when copied to the clipboard.

              Show
              Patrick Corless added a comment - I've done quite a bit of work on text selection as a whole. The main area of concern was with regards to missing selected text when mousing quickly. The new rules address the missing lines and in general make selection feel a little smoother. There are still a few corners cases where selection might seem unpredictable. This is mainly related to text ordering on a page. We attempt to sort text for a given page but there are cases such as columns were a full sort would actually break the ordering the selected text when copied to the clipboard.
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #51791 Thu Jul 27 13:42:36 MDT 2017 patrick.corless PDF-1103 improved multi-page text selection support, fixed bug where
              some text maybe skipped on quick mouse gestures.
              Files Changed
              Commit graph MODIFY /icepdf/branches/icepdf-6.2.0/icepdf/viewer/src/org/icepdf/ri/common/tools/TextSelectionViewHandler.java
              Commit graph MODIFY /icepdf/branches/icepdf-6.2.0/icepdf/viewer/src/org/icepdf/ri/common/tools/TextSelection.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #51808 Mon Jul 31 14:04:37 MDT 2017 patrick.corless PDF-1103 improved multi-page text selection support, fixed bug where
              some text maybe skipped on quick mouse gestures.
              Files Changed
              Commit graph MODIFY /icepdf/trunk/icepdf/viewer/viewer-awt/src/main/java/org/icepdf/ri/common/tools/TextSelectionViewHandler.java
              Commit graph MODIFY /icepdf/trunk/icepdf/viewer/viewer-awt/src/main/java/org/icepdf/ri/common/tools/TextSelection.java
              Hide
              Permalink
              Patrick Corless added a comment -

              Marking as fixed. Will reopen as needed.

              Show
              Patrick Corless added a comment - Marking as fixed. Will reopen as needed.
              Patrick Corless made changes -
              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:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: