eirikbakke commented on PR #7463:
URL: https://github.com/apache/netbeans/pull/7463#issuecomment-2256296908

   > Or we consider [JSVG](https://github.com/weisJ/jsvg) like FlatLaf and 
IntelliJ.
   
   Yes, that should be possible. Some visual comparisons would have to be made 
to ensure that the rendered SVGs look the same when rendered with a different 
library. (I assume the lighter libraries implement a smaller subset of the SVG 
standard than Batik.)
   
   > We need to consider whether Batik is now a platform API, or whether just 
the ability to load from an SVG is (with potential for minor incompatibilities 
in changing implementation).
   
   The SVG loader was intentionally made into an optional and pluggable 
ServiceProvider module, implementing org.openide.util.spi.SVGLoader in the 
openide.util.ui module. The current implementation, based on Batik, is in the 
optional openide.util.ui.svg module.
   
   > please put milestones on PRs and push people for review if necessary - I 
noticed a bunch of useful platform updates from you, like this, that were 
getting stale.
   
   OK, will do this more in the future.
   
   > The benefit of pushing this to NB24 is also that maybe we get a few more 
icons in to reduce the problem of 
https://github.com/apache/netbeans/pull/7463#issuecomment-2251774844 (followup 
PRs).
   
   I think there will always end up being a mix of old and new icons, due to 
the sheer number of them (it's not just the Test Results tab, but also Search 
Results, Notifications, Output + all the debugging ones etc.). But getting work 
gradually merged in makes it easier to do incremental work later.


-- 
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

Reply via email to