RFR: 8181084: JavaFX show big icons in system menu on macOS with Retina display [v5]
Kevin Rushforth
kcr at openjdk.org
Wed Aug 3 15:14:54 UTC 2022
On Mon, 25 Jul 2022 18:53:21 GMT, Paul <duke at openjdk.org> wrote:
>> Looks good (and tested on Mac). I have some comments.
>
> Thanks for all the feedback. I've updated as per your suggestions @jperedadnr and @kevinrushforth. Let me know how this looks to you - I'm learning a lot here :-)
>
> I ran the tests as per your example and got `802 tests completes, 28 failed, 108 skipped`, which is exactly the same as when I run them on the `master` branch. Is there something I should be doing on a Mac to get zero failures? (Although, it's a lot better than my Linux box which gets 200 failed!)
@gargoyle To clarify my inline comments, the minimal change you need to make is:
1. Change the tests of the four `jField` variables to remove the `> 0` (so it becomes a boolean test for non-null).
2. Change the tests of `width` and `height` to be `> 0` (rather than `> 1`)
My other suggestions are optional.
-------------
PR: https://git.openjdk.org/jfx/pull/743
More information about the openjfx-dev
mailing list