Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8166591

[macos 10.12] Trackpad scrolling of text on OS X 10.12 Sierra is very fast (Trackpad, Retina only)

    Details

    • Subcomponent:
    • Resolved In Build:
      team
    • OS:
      os_x

      Description

      Regression: no.
      Platform specific: Yes (OS X 10.12 with Retina display. Not reproducible on OS X 10.11 (with or without Retina), 10.12 without Retina display).
      It is reproducible only on trackpad. Mouse scrolling always works good.
      I tested JTextArea inside JScrollPane with 8u112, 8u75, 9b137 on OS X 10.12 with retina display and trackpad scrolls very fast.

      Scrolling of JPanel with different Swing components inside JScrollPane works good (by trackpad and mouse)
      Scrolling of text inside JTextArea inside JScrollPane works very fast when I use trackpad and works good when I use mouse.

        Issue Links

          Activity

          Show
          alexsch Alexander Scherbatiy added a comment - - edited The JDK 8u-dev backport request: http://mail.openjdk.java.net/pipermail/jdk8u-dev/2016-October/005985.html The JDK 8u-dev backport request: http://mail.openjdk.java.net/pipermail/awt-dev/2016-October/012087.html
          Hide
          arusakov Andrey Rusakov added a comment -
          Reproducible with 7u121b08, same scenario.
          Show
          arusakov Andrey Rusakov added a comment - Reproducible with 7u121b08, same scenario.
          Hide
          alexsch Alexander Scherbatiy added a comment -
          > There is manual regression test. Is there no automated test because neither robot, nor jemmy can not emulate sending a lot of small scroll values?

          Robot.mouseWheel(int wheelAmt) can generate only integer scroll values and there is no a method for trackpad emulation.
          Show
          alexsch Alexander Scherbatiy added a comment - > There is manual regression test. Is there no automated test because neither robot, nor jemmy can not emulate sending a lot of small scroll values? Robot.mouseWheel(int wheelAmt) can generate only integer scroll values and there is no a method for trackpad emulation.
          Hide
          alexsch Alexander Scherbatiy added a comment - - edited
          - webrev link: http://cr.openjdk.java.net/~alexsch/8166591/webrev-jdk8.00/
          - review link: in progress http://mail.openjdk.java.net/pipermail/awt-dev/2016-October/012088.html
          - JPRT build link: http://scaaa637.us.oracle.com//archive/2016/09/2016-09-30-171436.Scherbatiy.jdk8u-dev
          - issue impact: Small trackpad scrolling leads that a text is scrolled too fast on Mac OS X Sierra 10.12. There is no a way to scroll a text slowly by using the trackpad.
          - fix rational: The delta values from the trackpad are summarized until the exceed a certain threshold. After that a scroll event is set to 1.
           -risks: The fix allows to fire a MouseWheelEvent with zero wheelRotation value. It could have some problems for applications which does not expect this.
           -suggested testing: run the manual test: test/javax/swing/plaf/basic/BasicScrollPaneUI/8166591/TooMuchWheelRotationEventsTest.java
          Show
          alexsch Alexander Scherbatiy added a comment - - edited - webrev link: http://cr.openjdk.java.net/~alexsch/8166591/webrev-jdk8.00/ - review link: in progress http://mail.openjdk.java.net/pipermail/awt-dev/2016-October/012088.html - JPRT build link: http://scaaa637.us.oracle.com//archive/2016/09/2016-09-30-171436.Scherbatiy.jdk8u-dev - issue impact: Small trackpad scrolling leads that a text is scrolled too fast on Mac OS X Sierra 10.12. There is no a way to scroll a text slowly by using the trackpad. - fix rational: The delta values from the trackpad are summarized until the exceed a certain threshold. After that a scroll event is set to 1.  -risks: The fix allows to fire a MouseWheelEvent with zero wheelRotation value. It could have some problems for applications which does not expect this.  -suggested testing: run the manual test: test/javax/swing/plaf/basic/BasicScrollPaneUI/8166591/TooMuchWheelRotationEventsTest.java
          Hide
          afomin Alexander Fomin added a comment -
          UR SQE OK to take the fix to CPU17_01 for better support Mac OS 10.12.
          Please, backport it to jdk8.
          Do we need to get it backported to jdk7?
          Show
          afomin Alexander Fomin added a comment - UR SQE OK to take the fix to CPU17_01 for better support Mac OS 10.12. Please, backport it to jdk8. Do we need to get it backported to jdk7?

            People

            • Assignee:
              alexsch Alexander Scherbatiy
              Reporter:
              ampopov Alexander Popov
            • Votes:
              0 Vote for this issue
              Watchers:
              10 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: