ICEpdf
  1. ICEpdf
  2. PDF-1145

Form text area widget scaling is breaking text selection caret.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 6.2.3
    • Fix Version/s: 6.2.5
    • Component/s: Forms
    • Labels:
      None
    • Environment:
      PRO

      Description

      The client is using the text area component with scales != 100% and they are noticing inconsistencies with the caret location for the annotation text area component.

      The text field component seems to be working well enough, will have dig a little deeper to see what we can do to address this issue. P

        Activity

        Patrick Corless created issue -
        Patrick Corless made changes -
        Field Original Value New Value
        Fix Version/s 6.2.4 [ 13099 ]
        Judy Guglielmin made changes -
        Support Case References Support Case 14081:- https://icesoft.my.salesforce.com/5007000001j1Y8U
        Hide
        Patrick Corless added a comment -

        Overrided text area's processMouseMotionEventI() and processMouseEvent() methods and added in some custom scaling so the mouse events correctly map to the new zoom level.

        Show
        Patrick Corless added a comment - Overrided text area's processMouseMotionEventI() and processMouseEvent() methods and added in some custom scaling so the mouse events correctly map to the new zoom level.
        Hide
        Patrick Corless added a comment -

        Marking as fixed.

        Show
        Patrick Corless added a comment - Marking as fixed.
        Patrick Corless made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #51475 Mon May 08 09:52:46 MDT 2017 patrick.corless PDF-1145 fixed mouse coordinate issue when text area is not at 1000%
        zoom.
        Files Changed
        Commit graph MODIFY /icepdf/branches/icepdf-6.2.0/icepdf/viewer/src/org/icepdf/ri/common/views/annotations/ScalableTextArea.java
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #51476 Mon May 08 09:53:00 MDT 2017 patrick.corless PDF-1145 fixed mouse coordinate issue when text area is not at 1000%
        zoom.
        Files Changed
        Commit graph MODIFY /icepdf/trunk/icepdf/viewer/viewer-awt/src/main/java/org/icepdf/ri/common/views/annotations/ScalableTextArea.java
        Patrick Corless made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Hide
        Patrick Corless added a comment -

        It's better now, but the caret/cursor is slightly off-set.

        For example,

        1. Open the attached "N5_Mathematics_all_2016-DQP-Spellcheckoff.pdf"

        2. Maximize the window to a screen resolution of 1920x1080

        3. Select the "Town" field and type "aaaaabbbbbcccccdddddeeeee" (5 of each)

        4. Place the caret (with the keyboard arrow keys) after the last 'b', it
        will be in the middle of the next 'c', put caret after the last 'd' - it
        will be after 'e' (1-character offset each 20 symbols).

        It is more visible when you slowly select the text from the 1st 'a' to the
        last 'e'.

        added test case to support drive.

        We can reproduce the problem with the caret on the textarea as well as the text field widgets. There seems to be some inconsistencies in how the care bounds are calculated and pointed. Further debugging is needed.

        Show
        Patrick Corless added a comment - It's better now, but the caret/cursor is slightly off-set. For example, 1. Open the attached "N5_Mathematics_all_2016-DQP-Spellcheckoff.pdf" 2. Maximize the window to a screen resolution of 1920x1080 3. Select the "Town" field and type "aaaaabbbbbcccccdddddeeeee" (5 of each) 4. Place the caret (with the keyboard arrow keys) after the last 'b', it will be in the middle of the next 'c', put caret after the last 'd' - it will be after 'e' (1-character offset each 20 symbols). It is more visible when you slowly select the text from the 1st 'a' to the last 'e'. added test case to support drive. We can reproduce the problem with the caret on the textarea as well as the text field widgets. There seems to be some inconsistencies in how the care bounds are calculated and pointed. Further debugging is needed.
        Patrick Corless made changes -
        Fix Version/s 6.3 [ 13093 ]
        Patrick Corless made changes -
        Fix Version/s 6.2.5 [ 13102 ]
        Fix Version/s 6.3 [ 13093 ]
        Fix Version/s 6.2.4 [ 13099 ]
        Hide
        Patrick Corless added a comment -

        Marking as fixed.

        Show
        Patrick Corless added a comment - Marking as fixed.
        Patrick Corless made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #51662 Mon Jun 19 15:11:10 MDT 2017 patrick.corless PDF-1145 reword how we scale a text area component. This new approach
        seems to resolve a few issues with text carret as well as fixes an
        issue where some textfields had graphics artifacts from previous paints.
        Files Changed
        Commit graph MODIFY /icepdf/branches/icepdf-6.2.0/icepdf/viewer/src/org/icepdf/ri/common/views/annotations/FreeTextAnnotationComponent.java
        Commit graph MODIFY /icepdf/branches/icepdf-6.2.0/icepdf/viewer/src/org/icepdf/ri/common/views/annotations/ScalableTextArea.java
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #51663 Mon Jun 19 15:11:41 MDT 2017 patrick.corless PDF-1145 reword how we scale a text area component. This new approach
        seems to resolve a few issues with text carret as well as fixes an
        issue where some textfields had graphics artifacts from previous paints.
        Files Changed
        Commit graph MODIFY /icepdf/trunk/icepdf/viewer/viewer-awt/src/main/java/org/icepdf/ri/common/views/annotations/ScalableTextArea.java
        Commit graph MODIFY /icepdf/trunk/icepdf/viewer/viewer-awt/src/main/java/org/icepdf/ri/common/views/annotations/FreeTextAnnotationComponent.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: