[Rev 03] RFR: 8130738: Add tabSize property to Text and TextFlow

Kevin Rushforth kcr at openjdk.java.net
Sat Dec 14 16:37:40 UTC 2019


On Fri, 13 Dec 2019 01:10:48 GMT, Scott Palmer <swpalmer at openjdk.org> wrote:

>> modules/javafx.graphics/src/main/java/javafx/scene/text/Text.java line 1450:
>> 
>>> 1449:         private static final CssMetaData<Text,Number> TAB_SIZE =
>>> 1450:                 new CssMetaData<Text,Number>("-fx-tab-size",
>>> 1451:                 SizeConverter.getInstance(), TextLayout.DEFAULT_TAB_SIZE) {
>> 
>> I think that type parameters should have space separation after a comma: `Text, Number`, though I've seen without too.
> 
> Most of the instances where CssMetaData is used in the rest of the file don't have a space. The exceptions seem to be where there is a wildcard in the type parameter.  It could be left as-is simply to maintain consistency within the file.  Though I tend to agree that a space makes sense.

Either is fine with me in this case.

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

PR: https://git.openjdk.java.net/jfx/pull/32


More information about the openjfx-dev mailing list