eirikbakke commented on PR #8114: URL: https://github.com/apache/netbeans/pull/8114#issuecomment-2575380129
I pushed a revision to the two commits in this PR. Changes since your previous review: * Change the new loadImage(URL) to loadImage(URI). Update call sites accordingly in the second commit. * Clean up the Javadoc in ImageUtilities to avoid repeating the resource path loading semantics (substitution of dark mode suffixes and SVG images) for every similar method. Some other Javadoc edits for grammar, consistency etc. * Add the ImageUtilities.mergeIcons(Icon,Icon,int,int) method (like mergeImages(Image,Image,int,int) but takes and returns Icon instances instead of Image) as well, for future use. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] For further information about the NetBeans mailing lists, visit: https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists
