Hey,

I believe that on mac, the question icon and inform icon are simply the 
application icon.
This is the case for native alert dialogs, and this is also done in Java swing 
if your application is bundled correctly or when -Xdock:icon is specified via 
the command.

I am not sure if there is something that needs fixing.

The proposed fix seems like a step in the wrong direction

Koen

-----Original Message-----
From: swing-dev <swing-dev-r...@openjdk.java.net> On Behalf Of Prasanta 
Sadhukhan
Sent: Wednesday, July 15, 2020 10:25 AM
To: swing-dev@openjdk.java.net
Subject: <Swing Dev> RFR JDK-8210850: There is a Java icon instead of a i icon 
on mac

Hi All,

Please review a fix for an issue where it is seen JOptionPane showInputDialog 
and showConfirmationDialog in SwingSet2 "OptionPane Demo" dialog

is being shown as a "Java" icon instead of "i" icon for AquaLookAndFeel in mac. 
Other L&Fs show "i" icon.

Issue is because in Aqua L&F, JOptionPane.informationIcon and 
JOptionPane.questionIcon uses "confirmIcon" which uses generic java icon as 
seen here

https://urldefense.proofpoint.com/v2/url?u=http-3A__hg.openjdk.java.net_jdk_client_file_9078079d153b_src_java.desktop_macosx_classes_com_apple_laf_AquaImageFactory.java-23l53&d=DwICaQ&c=9mghv0deYPYDGP-W745IEdQLV1kHpn4XJRvR6xMRXtA&r=9d_eS2PJkOJltUfB0P4NB6MyXvE-X1oiMmv4r0btY2s&m=LSNvSA90f9CQ2ew0l5Nag5nXAoknapz0ote5Z8Tru58&s=giOu_dtf7JnXFrfZx5hNE1JqGQ2mwEaP4xZHWCcHG-o&e=

Proposed fix is to use appropriate icon gif image that is being used for other 
L&Fs for showInputDialog and showConfirmationDialog.

Other icons like errorIcon and warningIcon is not changed as it uses system 
icons via AquaIcon.SystemIcon icons but these system-icons are not present for 
the above 2 icons.

Bug: 
https://urldefense.proofpoint.com/v2/url?u=https-3A__bugs.openjdk.java.net_browse_JDK-2D8210850&d=DwICaQ&c=9mghv0deYPYDGP-W745IEdQLV1kHpn4XJRvR6xMRXtA&r=9d_eS2PJkOJltUfB0P4NB6MyXvE-X1oiMmv4r0btY2s&m=LSNvSA90f9CQ2ew0l5Nag5nXAoknapz0ote5Z8Tru58&s=Dw7UmI849E2lHvyXrHYzIKVY9QE-djJ_pyCPdh4Wiz8&e=

webrev: 
https://urldefense.proofpoint.com/v2/url?u=http-3A__cr.openjdk.java.net_-7Epsadhukhan_8210850_webrev.0_&d=DwICaQ&c=9mghv0deYPYDGP-W745IEdQLV1kHpn4XJRvR6xMRXtA&r=9d_eS2PJkOJltUfB0P4NB6MyXvE-X1oiMmv4r0btY2s&m=LSNvSA90f9CQ2ew0l5Nag5nXAoknapz0ote5Z8Tru58&s=6hel-l73633rQRiFvC0KMMyh9fw-zIVT3wqiUJ517Iw&e=

No regression test is provided as it can be verified through SwingSet2 
OptionPane demo.

Regards
Prasanta


Please be advised that this email may contain confidential information. If you 
are not the intended recipient, please notify us by email by replying to the 
sender and delete this message. The sender disclaims that the content of this 
email constitutes an offer to enter into, or the acceptance of, any agreement; 
provided that the foregoing does not invalidate the binding effect of any 
digital or other electronic reproduction of a manual signature that is included 
in any attachment.

Reply via email to