On Mon, 1 May 2023 19:16:31 GMT, Andy Goryachev <ango...@openjdk.org> wrote:
>> Monkey Tester - a JavaFX application designed to support manual ad-hoc >> testing of individual JavaFX controls. Unlike Ensemble, the goal of this >> application is to facilitate manual testing rather than demonstrate the >> capabilities of JavaFX. >> >> Feedback and suggestions are always welcome. >> >>  > > Andy Goryachev has updated the pull request incrementally with one additional > commit since the last revision: > > review comments tests/manual/monkey/src/com/oracle/tools/fx/monkey/pages/ComboBoxPage.java line 71: > 69: new Timeline( > 70: new KeyFrame(Duration.seconds(1.0), (ev) -> { > 71: System.out.println("2"); Why do we need that debug output in this particular place? May be add some label or text field that shows the currently assumed number of items instead of relying on the system output stream? Also it would be great if there is a text somewhere which at least describes what is the purpose of the button and what is supposed to happen when you press it. If not in some designated area then at least as a tooltip on the button? I mean "Change item count" - to what? No mention of the delay so user has to either guess of why does it not change immediately or look the source code. ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/1097#discussion_r1184138655
