On Fri, 3 Feb 2023 22:28:06 GMT, Kevin Rushforth <[email protected]> wrote:
>> Andy Goryachev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8301797: review comments > > modules/javafx.controls/src/main/java/javafx/scene/control/skin/PaginationSkin.java > line 197: > >> 195: nextStackPane.setVisible(false); >> 196: >> 197: resetIndexes(true); > > This will end up calling into the control in a couple places, which the PR > description says you are trying to avoid. true, but since it sets to the same value it will be a no-op ------------- PR: https://git.openjdk.org/jfx/pull/1021
