RFR: 6465404: some problems in CellEditor related API docs

Phil Race prr at openjdk.java.net
Mon Dec 6 17:24:12 UTC 2021


On Mon, 6 Dec 2021 16:29:03 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> JListBox is invalid component. CellTableEditor Interface doesn't support JComboBox and JTree components. Hence its removed from Documentation.
>
> src/java.desktop/share/classes/javax/swing/table/TableCellEditor.java line 35:
> 
>> 33:  * This interface defines the method any object that would like to be
>> 34:  * an editor of values for the component <code>JTable</code> that
>> 35:  * needs to implement.
> 
> Suggestion:
> 
>  * an editor of values for the {@code JTable} component needs to implement.
> 
> _‘that’_ after `JTable` shouldn't be there.
> 
> However, I would suggest rephrasing the Javadoc to make it easier to understand: the sentence has two levels of dependent clauses.

I'm not sure this fix is going in the right direction. Isn't the doc saying that if you want to use one of these components as the cell of a JTable it should implement this interface ?
And the problem with JListBox is that it is actually just called JList ?

-------------

PR: https://git.openjdk.java.net/jdk/pull/6608



More information about the client-libs-dev mailing list