Probably we can update the BasicTreeUI.setLargeModel() as well? current spec
looks incomplete:
/**
* Updates the componentListener, if necessary.
*
* @param largeModel the new value
*/
protected void setLargeModel(boolean largeModel) {
On 12/8/19 11:53 pm, Tejpal Rebari wrote:
Gentle reminder....
On 04-Dec-2019, at 3:02 PM, Tejpal Rebari <[email protected]
<mailto:[email protected]>> wrote:
Hi Phil,
On 27-Nov-2019, at 9:59 PM, Philip Race <[email protected]
<mailto:[email protected]>> wrote:
Yes it needs a CSR.
Will raise a CSR once the review is done.
The updated wording explains when the "set*" methods are used/called but does
not do the same for the get* methods. Why not for both ?
The setters will be called when the property (like rowHeight, cellRenderer
etc.) changes in the drawn tree component.
But the getters won’t be called when the property changes, so they need not to
be updated.
Regards
Tejpal
-phil.
On 11/24/19, 11:31 PM, Jayathirth Rao wrote:
Its looks okay.
Lets wait for Sergey’s inputs on the same.
And I think it needs CSR also.
Thanks,
Jay
On 22-Nov-2019, at 3:04 PM, Tejpal Rebari <[email protected]
<mailto:[email protected]>> wrote:
Hi Jay,
I have appended the text instead of replacing it,
and also updated the SelectionModel methods.
Please find the updated webrev :
http://cr.openjdk.java.net/~trebari/swing/7020860/webrev1/
<http://cr.openjdk.java.net/%7Etrebari/swing/7020860/webrev1/>
Thanks
Tejpal
On 18-Nov-2019, at 1:00 PM, Jayathirth D V <[email protected]
<mailto:[email protected]>> wrote:
Hi Tejpal,
I think Sergey has given more inputs on the present change that you have at :
http://mail.openjdk.java.net/pipermail/swing-dev/2016-October/006815.html
andhttp://mail.openjdk.java.net/pipermail/swing-dev/2017-June/007495.html
Apart from this changes are fine.
Thanks,
Jay
*From:*Tejpal Rebari
*Sent:*Thursday, October 31, 2019 1:55 PM
*To:*[email protected] <mailto:[email protected]>
*Subject:*<Swing Dev> [14] RFR JDK-7020860 - BasicTreeUI contains
getters/setters with unclear spec
Hi All,
Please review the following fix for JDK 14.
Bug : https://bugs.openjdk.java.net/browse/JDK-7020860
Webrev : http://cr.openjdk.java.net/~trebari/swing/7020860/webrev0/
<http://cr.openjdk.java.net/%7Etrebari/swing/7020860/webrev0/>
Issue : Class javax.swing.plaf.basic.BasicTreeUI contains some getters/setters
that are not consistent and
and have specification that says nothing about how they should be used and how
do they behave in BasicTreeUI.
Fix : This bug was already sent for review by semyon
http://cr.openjdk.java.net/~ssadetsky/7020860/webrev.00/
<http://cr.openjdk.java.net/%7Essadetsky/7020860/webrev.00/>.
I have made the following changes to the above webrev :
* @param newValue the new value of the {@code editable}> property
changed to —>>
* @param newValue the new value of the {@code editable} property
And
* Returns whether the root node of the drawn tree component is displayable.
changed to —>>
* Returns whether the root node of the drawn tree component should be displayed.
Regards
Tejpal
--
Best regards, Sergey.
