RFR: 8181084: JavaFX show big icons in system menu on macOS with Retina display [v5]
Paul
duke at openjdk.org
Mon Jul 25 18:57:20 UTC 2022
On Mon, 25 Jul 2022 11:40:33 GMT, Jose Pereda <jpereda at openjdk.org> wrote:
>> Paul has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains seven additional commits since the last revision:
>>
>> - Merge branch 'master' into JDK-8181084
>> - Merge branch 'master' into JDK-8181084
>> - Updated variable names to be a bit more consistent
>> - Merge branch 'master' into JDK-8181084
>> - Merge branch 'master' into JDK-8181084
>> - Removing trailing whitespace.
>> - Correctly scales hi-res icons in MacOS system menu items
>
> 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!)
-------------
PR: https://git.openjdk.org/jfx/pull/743
More information about the openjfx-dev
mailing list