neilcsmith-net commented on PR #7566:
URL: https://github.com/apache/netbeans/pull/7566#issuecomment-2227990711

   Actually, looking at this afresh on my computer rather than trying to make 
sense of the test trace on my phone, I realised the dependency on the Window 
API is there.  The problem seems to be the lack of a window manager token.  I'm 
pretty sure this isn't required, but was added by the APISupport TopComponent 
wizard and I hadn't realised.
   
   See 
https://github.com/apache/netbeans/blob/master/apisupport/apisupport.wizards/src/org/netbeans/modules/apisupport/project/ui/wizard/winsys/NewTCIterator.java#L321
   
   Whereas the window API arch docs say "You might use 
`OpenIDE-Module-Requires: org.openide.windows.WindowManager` but it is not 
generally done. "
   
   Longer term we could if necessary move the default (Swing) dashboard 
displayer into a separate module.  But hopefully this fixes the immediate 
issue.  @sdedic the API itself doesn't rely on Swing, aside from actions, so an 
alternative displayer could be used in the VSCode plugin in future, should 
there be interest in exposing any widgets added.


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