On Thu, 3 Oct 2024 07:49:22 GMT, Alexey Ivanov <[email protected]> wrote:
> The default title for the instruction frame in `PassFailJFrame` is currently > _“Test Instruction Frame”_. > > Most tests change this default to _“Test Instructions”_ which describes the > content and purpose of the frame better. So, this should be the default > title, and it's shorter. > > jtreg has a property which describes the test name. Thus, the title could be > more specific: _“<testName.java> - Test Instructions”_. > > Additionally, I added javadoc comments to the default parameters of > `PassFailJFrame` and to the `Builder.validate` method. Marked as reviewed by prr (Reviewer). ------------- PR Review: https://git.openjdk.org/jdk/pull/21320#pullrequestreview-2346559552
