On Thu, 16 May 2024 19:16:26 GMT, Phil Race <[email protected]> wrote:
>> Enhance Font2DTest as follows
>>
>> - Add main menu Radio Button options so that you select the font to use as
>> either
>> - (1) Font Family + Style (as now)
>> - (2) Font Family + Menu of all members of the Family, replacing the Style
>> - (3) List of all fontnames - which can still be adjusted by Style if you
>> want.
>> The default is (1) so nothing looks different except that I updated the UI
>> to use Nimbus instead of Metal.
>>
>> There's new code to gather these ways of referencing the fonts.
>> Also changes were needed for the "Save/Load" options to include the new UI
>> state and font settings.
>
> Phil Race has updated the pull request incrementally with one additional
> commit since the last revision:
>
> 8332416
src/demo/share/jfc/Font2DTest/Font2DTest.java line 1273:
> 1271: familyIndex++;
> 1272: }
> 1273: else if (fullChars[fullIndex] == ' ' ||
> fullChars[fullIndex] == '-') {
Suggestion:
else if (fullChars[fullIndex] == ' ' || fullChars[fullIndex] ==
'-') {
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19273#discussion_r1608639918
