RFR: 8328402: Implement pausing functionality for the PassFailJFrame [v2]
Alexander Zvegintsev
azvegint at openjdk.org
Wed Mar 20 20:32:21 UTC 2024
On Wed, 20 Mar 2024 17:58:59 GMT, Alexey Ivanov <aivanov at 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
More information about the client-libs-dev
mailing list