You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pivot.apache.org by "Greg Brown (JIRA)" <ji...@apache.org> on 2009/07/30 22:38:14 UTC

[jira] Created: (PIVOT-197) Clean up multiple selection implementation

Clean up multiple selection implementation
------------------------------------------

                 Key: PIVOT-197
                 URL: https://issues.apache.org/jira/browse/PIVOT-197
             Project: Pivot
          Issue Type: Improvement
          Components: wtk
    Affects Versions: 1.3
            Reporter: Greg Brown
            Assignee: Greg Brown
            Priority: Minor
             Fix For: 1.3


Convert Span to a "struct"; replace SpanSequence with ListSelection and ListSelectionSequence. Return an instance of ListSelectionSequence from ListView#getSelectedRanges() and TableView#getSelectedRanges(). This will avoid the need to copy the selection contents. Fire selected range add and remove events only for ranges that are actually added and removed (don't re-fire for existing selections).



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


[jira] Resolved: (PIVOT-197) Clean up multiple selection implementation

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

Greg Brown resolved PIVOT-197.
------------------------------

    Resolution: Fixed

> Clean up multiple selection implementation
> ------------------------------------------
>
>                 Key: PIVOT-197
>                 URL: https://issues.apache.org/jira/browse/PIVOT-197
>             Project: Pivot
>          Issue Type: Improvement
>          Components: wtk
>    Affects Versions: 1.3
>            Reporter: Greg Brown
>            Assignee: Greg Brown
>            Priority: Minor
>             Fix For: 1.3
>
>
> Convert Span to a "struct"; replace SpanSequence with ListSelection and ListSelectionSequence. Return an instance of ListSelectionSequence from ListView#getSelectedRanges() and TableView#getSelectedRanges(). This will avoid the need to copy the selection contents. Fire selected range add and remove events only for ranges that are actually added and removed (don't re-fire for existing selections).

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


[jira] Commented: (PIVOT-197) Clean up multiple selection implementation

Posted by "Greg Brown (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/PIVOT-197?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12737810#action_12737810 ] 

Greg Brown commented on PIVOT-197:
----------------------------------

Span has been converted to a "struct"; SpanSequence has been replaced with ListSelection and ListSelectionSequence. Events are still being fired for the entire range, not just the ranges that were modified - this will be addressed in the next update.


> Clean up multiple selection implementation
> ------------------------------------------
>
>                 Key: PIVOT-197
>                 URL: https://issues.apache.org/jira/browse/PIVOT-197
>             Project: Pivot
>          Issue Type: Improvement
>          Components: wtk
>    Affects Versions: 1.3
>            Reporter: Greg Brown
>            Assignee: Greg Brown
>            Priority: Minor
>             Fix For: 1.3
>
>
> Convert Span to a "struct"; replace SpanSequence with ListSelection and ListSelectionSequence. Return an instance of ListSelectionSequence from ListView#getSelectedRanges() and TableView#getSelectedRanges(). This will avoid the need to copy the selection contents. Fire selected range add and remove events only for ranges that are actually added and removed (don't re-fire for existing selections).

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


[jira] Commented: (PIVOT-197) Clean up multiple selection implementation

Posted by "Greg Brown (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/PIVOT-197?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12737880#action_12737880 ] 

Greg Brown commented on PIVOT-197:
----------------------------------

This change is complete but requires additional unit testing.


> Clean up multiple selection implementation
> ------------------------------------------
>
>                 Key: PIVOT-197
>                 URL: https://issues.apache.org/jira/browse/PIVOT-197
>             Project: Pivot
>          Issue Type: Improvement
>          Components: wtk
>    Affects Versions: 1.3
>            Reporter: Greg Brown
>            Assignee: Greg Brown
>            Priority: Minor
>             Fix For: 1.3
>
>
> Convert Span to a "struct"; replace SpanSequence with ListSelection and ListSelectionSequence. Return an instance of ListSelectionSequence from ListView#getSelectedRanges() and TableView#getSelectedRanges(). This will avoid the need to copy the selection contents. Fire selected range add and remove events only for ranges that are actually added and removed (don't re-fire for existing selections).

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