On Wed, 20 Mar 2024 17:58:59 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:
> I would place the Pause button into the timeout panel at the top. Yet the > button is larger than the label. The font may be reduced, a symbol '⏸' > (U+23F8) and '⏵' (U+23F5) could be used (see [Media control > symbols](https://en.wikipedia.org/wiki/Media_control_symbols)), or an image > icon with a tooltip. Unfortunately `a symbol '⏸' (U+23F8) and '⏵' (U+23F5)` doesn't work on Linux for me:   So the text is used (icons can be added separately):       > test/jdk/java/awt/regtesthelpers/PassFailJFrame.java line 704: > >> 702: } else { >> 703: >> label.setText(label.getText().replace(PAUSED_LABEL_SUFFIX, "")); >> 704: endTime = System.currentTimeMillis() + pauseTimeLeft; > > I'd rather call `updateTime(pauseTimeLeft)` here instead of editing the text. > > Suggestion: > > endTime = System.currentTimeMillis() + pauseTimeLeft; > updateTime(pauseTimeLeft); Yes, this way is better. ------------- PR Comment: https://git.openjdk.org/jdk/pull/18368#issuecomment-2010561001 PR Review Comment: https://git.openjdk.org/jdk/pull/18368#discussion_r1532807064