RFR: 8355203: [macos] AquaButtonUI and AquaRootPaneUI repaint default button unnecessarily [v3]
Jeremy Wood
duke at openjdk.org
Fri Apr 25 07:49:54 UTC 2025
> This removes the constant repainting of the AquaRootPaneUI's default button.
>
> A long time ago (before Mac OS Yosemite, around 2014) this was needed to match Aqua's native experience. But the default button no longer pulses, so we don't need to repaint ("pulse") the button continually.
>
> Here's a clip from a YouTube video I found demonstrating the pulsing effect that this PR removes (you may want to turn your volume off):
>
> https://github.com/user-attachments/assets/496bc003-ccba-4178-81bc-765e96d94316
>
> This PR includes a test, but this test has never failed. (That is: this PR does not resolve a failing test.) This test would fail if the default button ever looked the same as a non-default button.
>
> My primary goal here was to remove the AquaButtonUI's AncestorListener, but while we're here we might as well remove the AquaRootPaneUI's timer and other cleanup.
>
> I tried and failed to rename the `private static final byte _pulsed` constant. (More details [here](https://github.com/openjdk/jdk/commit/89d0689f226b9e883df464e6f3e8c20074970499).) If anyone has suggestions for how to rename that without errors please let me know.
Jeremy Wood has updated the pull request incrementally with two additional commits since the last revision:
- 8355203: breaking up long line into multiple lines
This is in response to:
https://github.com/openjdk/jdk/pull/24778#discussion_r2059701077
- 8355203: breaking up long line into multiple lines
This is in response to:
https://github.com/openjdk/jdk/pull/24778#discussion_r2059701077
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/24778/files
- new: https://git.openjdk.org/jdk/pull/24778/files/60021fcc..562804e5
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=24778&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=24778&range=01-02
Stats: 37 lines in 1 file changed: 15 ins; 0 del; 22 mod
Patch: https://git.openjdk.org/jdk/pull/24778.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/24778/head:pull/24778
PR: https://git.openjdk.org/jdk/pull/24778
More information about the client-libs-dev
mailing list