<Swing Dev> RFR: 8268084: [macos] Disabled JMenuItem arrow is not disabled [v4]
Prasanta Sadhukhan
psadhukhan at openjdk.java.net
Fri Sep 3 07:50:03 UTC 2021
> It is seen in macos disabled JMenuItem arrow is not disabled even though JMenuItem itself is disabled.
> In native app, same menuitem arrow is disabled for disabled menuitem.
>
> Issue is when AquaMenuPainter#paintMenuItem() is called, it tries to draw a ImageIcon image of the arrow via ImageIcon#paintIcon which tries to generate MultiResolutionCachedImage via getResolutionVariant() by calling AquaUtils#generateFilteredImage.
> It does not take into account if disabled arrow icon image needs to be drawn or not, so it is always enabled.
>
> Proposed fix is to generate a disabled ImageIcon image of the same arrow icon and use it for disabled state.
Prasanta Sadhukhan has updated the pull request incrementally with two additional commits since the last revision:
- Override paintIcon
- Override paintIcon
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/5310/files
- new: https://git.openjdk.java.net/jdk/pull/5310/files/63b591cc..d2db8808
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=5310&range=03
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=5310&range=02-03
Stats: 3 lines in 1 file changed: 0 ins; 3 del; 0 mod
Patch: https://git.openjdk.java.net/jdk/pull/5310.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/5310/head:pull/5310
PR: https://git.openjdk.java.net/jdk/pull/5310
More information about the swing-dev
mailing list