Uploaded image for project: 'Pivot'
  1. Pivot
  2. PIVOT-923

ScrollBar not displayed correctly if start is greater than 0

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.3
    • Fix Version/s: 2.0.4, 2.1, 2.0.5
    • Component/s: wtk-terra
    • Labels:
      None

      Description

      The TerraScrollBarSkin does not checks the start value of the underlying ScrollBar on some operations

      1. patch-scrollbar-2.0.4.diff
        0.5 kB
        Alessandro Pira
      2. patch-scrollbar-2.0.3.diff
        3 kB
        Alessandro Pira

        Activity

        Hide
        alessandro.pira Alessandro Pira added a comment -

        Looks fine to me. Thank you!

        Show
        alessandro.pira Alessandro Pira added a comment - Looks fine to me. Thank you!
        Hide
        smartini Sandro Martini added a comment -

        For 2.0.x: Committed revision 1611025.
        Alessandro, please could you make some test with latest (committed) sources, to ensure that this time all has been fixed (I hope , and mark as resolved) ? Thank you very much.
        I'll merge the fix even in trunk now.

        Show
        smartini Sandro Martini added a comment - For 2.0.x: Committed revision 1611025. Alessandro, please could you make some test with latest (committed) sources, to ensure that this time all has been fixed (I hope , and mark as resolved) ? Thank you very much. I'll merge the fix even in trunk now.
        Hide
        alessandro.pira Alessandro Pira added a comment -

        No problem. Thank you very much, Sandro.

        Show
        alessandro.pira Alessandro Pira added a comment - No problem. Thank you very much, Sandro.
        Hide
        smartini Sandro Martini added a comment -

        It seems that part of the fix has been lost, so now I have to apply it both to 2.0.x and trunk ... sorry for this.

        Show
        smartini Sandro Martini added a comment - It seems that part of the fix has been lost, so now I have to apply it both to 2.0.x and trunk ... sorry for this.
        Hide
        alessandro.pira Alessandro Pira added a comment -

        It seems that 2.0.4 is missing part of the patch. I am attaching the fix we had to apply to 2.0.4.

        Show
        alessandro.pira Alessandro Pira added a comment - It seems that 2.0.4 is missing part of the patch. I am attaching the fix we had to apply to 2.0.4.
        Hide
        smartini Sandro Martini added a comment -

        I'll merge the patch even in trunk (for 2.1), soon.

        Show
        smartini Sandro Martini added a comment - I'll merge the patch even in trunk (for 2.1), soon.
        Hide
        smartini Sandro Martini added a comment - - edited

        Patch applied (but by hand, probably is was not in svn format) and tested with some applications ... thanks for now.

        Show
        smartini Sandro Martini added a comment - - edited Patch applied (but by hand, probably is was not in svn format) and tested with some applications ... thanks for now.
        Hide
        alessandro.pira Alessandro Pira added a comment -

        I'm attaching the patch I used to fix the problem.

        Show
        alessandro.pira Alessandro Pira added a comment - I'm attaching the patch I used to fix the problem.

          People

          • Assignee:
            smartini Sandro Martini
            Reporter:
            alessandro.pira Alessandro Pira
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development