> 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