<Swing Dev> RFR: 8182043: Access to Windows Large Icons
Alexander Zuev
kizune at openjdk.java.net
Thu Apr 29 14:52:54 UTC 2021
On Fri, 2 Apr 2021 21:20:49 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:
>> Fix updated after first round of review.
>
> src/java.desktop/share/classes/javax/swing/filechooser/FileSystemView.java line 276:
>
>> 274: *
>> 275: * @param f a <code>File</code> object
>> 276: * @param size width and height of the icon in pixels
>
> Pixels could be ambiguous now. Usually, Swing deals with user's space. That is 16×16 icon should actually be 32×32 if the scale factor of the current display is 200%.
>
> Yes, this issue is somewhat irrelevant because the method returns multi-resolution icon. However, the terminology used must be unambiguous and clear; for consistency with other Swing API, it should be in terms of user's space coordinates, *virtual pixels*.
Ok, fixed.
-------------
PR: https://git.openjdk.java.net/jdk/pull/2875
More information about the swing-dev
mailing list