You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pivot.apache.org by "Todd Volkert (JIRA)" <ji...@apache.org> on 2009/08/31 01:47:32 UTC

[jira] Created: (PIVOT-255) Possible rounding error in TerraTableViewSkin.paint()

Possible rounding error in TerraTableViewSkin.paint()
-----------------------------------------------------

                 Key: PIVOT-255
                 URL: https://issues.apache.org/jira/browse/PIVOT-255
             Project: Pivot
          Issue Type: Bug
          Components: wtk
    Affects Versions: 1.3
         Environment: Verified in Windows and Linux - Mac OSX not tested
            Reporter: Todd Volkert
            Priority: Trivial
             Fix For: 1.3.2


1) Run org.apache.pivot.demos.tables.FixedColumnTable
2) Select "User 6" row
3) Move your mouse between the "User 5" and "User 7" rows

Result:
As you move your mouse above and below the selected row, columns a, b, and c toggle an extra row of pixels painted selected.  The extra row of pixels of very subtle, but it's noticeable when you follow the steps above.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[jira] Updated: (PIVOT-255) Possible rounding error in TerraTableViewSkin.paint()

Posted by "Todd Volkert (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/PIVOT-255?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Todd Volkert updated PIVOT-255:
-------------------------------

    Attachment: above-yes-extra-pixel.png
                below-no-extra-pixel.png

> Possible rounding error in TerraTableViewSkin.paint()
> -----------------------------------------------------
>
>                 Key: PIVOT-255
>                 URL: https://issues.apache.org/jira/browse/PIVOT-255
>             Project: Pivot
>          Issue Type: Bug
>          Components: wtk
>    Affects Versions: 1.3
>         Environment: Verified in Windows and Linux - Mac OSX not tested
>            Reporter: Todd Volkert
>            Priority: Trivial
>             Fix For: 1.3.2
>
>         Attachments: above-yes-extra-pixel.png, below-no-extra-pixel.png
>
>
> 1) Run org.apache.pivot.demos.tables.FixedColumnTable
> 2) Select "User 6" row
> 3) Move your mouse between the "User 5" and "User 7" rows
> Result:
> As you move your mouse above and below the selected row, columns a, b, and c toggle an extra row of pixels painted selected.  The extra row of pixels of very subtle, but it's noticeable when you follow the steps above.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[jira] Commented: (PIVOT-255) Possible rounding error in TerraTableViewSkin.paint()

Posted by "Todd Volkert (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/PIVOT-255?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12749349#action_12749349 ] 

Todd Volkert commented on PIVOT-255:
------------------------------------

A variation of this has been confirmed to happen on the Mac.  When you first select the row, the extra pixel is not there.  When you move the mouse above the row, the extra pixel shows up, and it stays there even when you move the mouse below the row.

> Possible rounding error in TerraTableViewSkin.paint()
> -----------------------------------------------------
>
>                 Key: PIVOT-255
>                 URL: https://issues.apache.org/jira/browse/PIVOT-255
>             Project: Pivot
>          Issue Type: Bug
>          Components: wtk
>    Affects Versions: 1.3
>         Environment: Verified in Windows and Linux - Mac OSX not tested
>            Reporter: Todd Volkert
>            Priority: Trivial
>             Fix For: 1.3.2
>
>         Attachments: above-yes-extra-pixel.png, below-no-extra-pixel.png
>
>
> 1) Run org.apache.pivot.demos.tables.FixedColumnTable
> 2) Select "User 6" row
> 3) Move your mouse between the "User 5" and "User 7" rows
> Result:
> As you move your mouse above and below the selected row, columns a, b, and c toggle an extra row of pixels painted selected.  The extra row of pixels of very subtle, but it's noticeable when you follow the steps above.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[jira] Updated: (PIVOT-255) Possible rounding error in TerraTableViewSkin.paint()

Posted by "Todd Volkert (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/PIVOT-255?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Todd Volkert updated PIVOT-255:
-------------------------------

    Fix Version/s:     (was: 1.3.1)
                   1.3

> Possible rounding error in TerraTableViewSkin.paint()
> -----------------------------------------------------
>
>                 Key: PIVOT-255
>                 URL: https://issues.apache.org/jira/browse/PIVOT-255
>             Project: Pivot
>          Issue Type: Bug
>          Components: wtk
>    Affects Versions: 1.3
>         Environment: Verified in Windows and Linux - Mac OSX not tested
>            Reporter: Todd Volkert
>            Assignee: Greg Brown
>            Priority: Trivial
>             Fix For: 1.3
>
>         Attachments: above-yes-extra-pixel.png, below-no-extra-pixel.png
>
>
> 1) Run org.apache.pivot.demos.tables.FixedColumnTable
> 2) Select "User 6" row
> 3) Move your mouse between the "User 5" and "User 7" rows
> Result:
> As you move your mouse above and below the selected row, columns a, b, and c toggle an extra row of pixels painted selected.  The extra row of pixels of very subtle, but it's noticeable when you follow the steps above.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[jira] Resolved: (PIVOT-255) Possible rounding error in TerraTableViewSkin.paint()

Posted by "Greg Brown (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/PIVOT-255?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Greg Brown resolved PIVOT-255.
------------------------------

       Resolution: Fixed
    Fix Version/s:     (was: 1.3.2)
                   1.3.1
         Assignee: Greg Brown

This wasn't actually a rounding error - it was introduced by a recent fix to how rows are painted.

This fix changes the definition of "row height" to include only the cell height, not the cell height plus the height of a horizontal grid line. This is now consistent with how column widths are defined.


> Possible rounding error in TerraTableViewSkin.paint()
> -----------------------------------------------------
>
>                 Key: PIVOT-255
>                 URL: https://issues.apache.org/jira/browse/PIVOT-255
>             Project: Pivot
>          Issue Type: Bug
>          Components: wtk
>    Affects Versions: 1.3
>         Environment: Verified in Windows and Linux - Mac OSX not tested
>            Reporter: Todd Volkert
>            Assignee: Greg Brown
>            Priority: Trivial
>             Fix For: 1.3.1
>
>         Attachments: above-yes-extra-pixel.png, below-no-extra-pixel.png
>
>
> 1) Run org.apache.pivot.demos.tables.FixedColumnTable
> 2) Select "User 6" row
> 3) Move your mouse between the "User 5" and "User 7" rows
> Result:
> As you move your mouse above and below the selected row, columns a, b, and c toggle an extra row of pixels painted selected.  The extra row of pixels of very subtle, but it's noticeable when you follow the steps above.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.