Changes between Initial Version and Version 1 of Ticket #15403, comment 8


Ignore:
Timestamp:
Jun 1, 2012, 5:41:38 PM (9 years ago)
Author:
Adrian Vasiliu
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #15403, comment 8

    initial v1  
    11I reproduce on Chrome 19 on Vista (Thinkpad), both the vertical position issue, and the "Reset" button having no effect. I do not reproduce on Firefox 12, nor on a few iPhone or Android devices that I've tried. It seems the problem hurts or not depending on number rounding issues, and maybe on timing/speed issues.
    22
    3 At least for the problem of the "Reset" button/method, the cause appears to be that !SpinWheelSlot.spin() has a numeric test to determine whether the spinning animation is ongoing, and this test fails because, in webkit, scrolling.getPos() returns sometimes a "y" value with decimals... The attached patch is a simple fix of this issue. Note that this patch only fixes the Reset issue, not the vertical position issue.
     3At least for the problem of the "Reset" button/method, the cause appears to be that !SpinWheelSlot.spin() has a numeric test to determine whether the spinning animation is ongoing, and this test fails because, in webkit, scrolling.getPos() returns sometimes a "y" value with decimals... The attached patch15403-reset.patch is a simple fix of this issue. Note that this patch only fixes the Reset issue, not the vertical position issue.