RFR: 8340308: PassFailJFrame: Make rows default to number of lines in instructions
Alexander Zvegintsev
azvegint at openjdk.org
Thu Sep 19 15:13:36 UTC 2024
On Tue, 17 Sep 2024 13:45:06 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:
> It has become quite common to use the following code
>
>
> .rows((int) INSTRUCTIONS.lines().count() + 1)
>
>
> to set the number of rows for `PassFailJFrame`.
>
> Make this the default value. Newer tests could drop `.rows` from builder configuration if the new default works well.
>
> If a test needs customising the new default value, the newly added method `rowsAdd` could be used to increase the number of rows.
As another improvement, we can also consider calculating the default number of columns (based on the widest string?) in a separate issue.
-------------
Marked as reviewed by azvegint (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/21039#pullrequestreview-2315852047
More information about the client-libs-dev
mailing list