RFR: JDK-8245919: Region#padding property rendering error [v3]

Kevin Rushforth kcr at openjdk.org
Mon May 15 18:25:50 UTC 2023


On Mon, 15 May 2023 17:23:45 GMT, John Hendrikx <jhendrikx at openjdk.org> wrote:

>> modules/javafx.graphics/src/main/java/javafx/scene/CssStyleHelper.java line 1185:
>> 
>>> 1183:         if (cssValue != null && "inherit".equals(cssValue.getValue())) {
>>> 1184:             style = getInheritedStyle(styleable, property);
>>> 1185:             if (style == null) return SKIP;
>> 
>> minor: could we surround return with { }'s please?  To keep one statement per line.
>
> It's not my code originally, I only changed the indent here.

Agreed. This code is otherwise unchanged, so I'd prefer it left as is.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1072#discussion_r1194196694


More information about the openjfx-dev mailing list