On Mon, 15 May 2023 17:23:45 GMT, John Hendrikx <jhendr...@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