[jira] [Commented] (PIVOT-763) Add tri-state checkbox renderer for TableView row editors

2011-06-22 Thread Roger Whitcomb (JIRA)
[ https://issues.apache.org/jira/browse/PIVOT-763?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13053294#comment-13053294 ] Roger Whitcomb commented on PIVOT-763: -- Committed revision 1138493. > Add tri-state c

[jira] [Commented] (PIVOT-763) Add tri-state checkbox renderer for TableView row editors

2011-06-22 Thread Roger Whitcomb (JIRA)
[ https://issues.apache.org/jira/browse/PIVOT-763?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13053293#comment-13053293 ] Roger Whitcomb commented on PIVOT-763: -- Committed revision 1138493. > Add tri-state c

[jira] [Commented] (PIVOT-763) Add tri-state checkbox renderer for TableView row editors

2011-06-21 Thread Sandro Martini (JIRA)
[ https://issues.apache.org/jira/browse/PIVOT-763?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13052923#comment-13052923 ] Sandro Martini commented on PIVOT-763: -- Hi Roger, if you commit (of course in the trun

[jira] [Commented] (PIVOT-763) Add tri-state checkbox renderer for TableView row editors

2011-06-21 Thread Roger Whitcomb (JIRA)
[ https://issues.apache.org/jira/browse/PIVOT-763?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13052912#comment-13052912 ] Roger Whitcomb commented on PIVOT-763: -- Thanks. I will tonight. > Add tri-state chec

[jira] [Commented] (PIVOT-763) Add tri-state checkbox renderer for TableView row editors

2011-06-21 Thread Greg Brown (JIRA)
[ https://issues.apache.org/jira/browse/PIVOT-763?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13052906#comment-13052906 ] Greg Brown commented on PIVOT-763: -- Looks good to me. You have commit rights now, right? F

[jira] [Commented] (PIVOT-763) Add tri-state checkbox renderer for TableView row editors

2011-06-21 Thread Greg Brown (JIRA)
[ https://issues.apache.org/jira/browse/PIVOT-763?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13052886#comment-13052886 ] Greg Brown commented on PIVOT-763: -- I think ideally we could just get away with the one T

[jira] [Commented] (PIVOT-763) Add tri-state checkbox renderer for TableView row editors

2011-06-21 Thread Roger Whitcomb (JIRA)
[ https://issues.apache.org/jira/browse/PIVOT-763?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13052884#comment-13052884 ] Roger Whitcomb commented on PIVOT-763: -- Do you mean making a base class TableViewCheck

[jira] [Commented] (PIVOT-763) Add tri-state checkbox renderer for TableView row editors

2011-06-21 Thread Greg Brown (JIRA)
[ https://issues.apache.org/jira/browse/PIVOT-763?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13052881#comment-13052881 ] Greg Brown commented on PIVOT-763: -- Using load() seems reasonable. It is also used by the

[jira] [Commented] (PIVOT-763) Add tri-state checkbox renderer for TableView row editors

2011-06-21 Thread Roger Whitcomb (JIRA)
[ https://issues.apache.org/jira/browse/PIVOT-763?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13052879#comment-13052879 ] Roger Whitcomb commented on PIVOT-763: -- The other thing I thought about was only chang

[jira] [Commented] (PIVOT-763) Add tri-state checkbox renderer for TableView row editors

2011-06-21 Thread Roger Whitcomb (JIRA)
[ https://issues.apache.org/jira/browse/PIVOT-763?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13052875#comment-13052875 ] Roger Whitcomb commented on PIVOT-763: -- That was the way I originally implemented it,

[jira] [Commented] (PIVOT-763) Add tri-state checkbox renderer for TableView row editors

2011-06-21 Thread Greg Brown (JIRA)
[ https://issues.apache.org/jira/browse/PIVOT-763?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13052872#comment-13052872 ] Greg Brown commented on PIVOT-763: -- I like the concept, but I wonder if TableViewBooleanCe