matthiasblaesing commented on PR #7938: URL: https://github.com/apache/netbeans/pull/7938#issuecomment-2555829153
For problem 1, this is the solution in `ide/image`: https://github.com/apache/netbeans/blob/9d5bc1e080daeac06ce0af3e6f1130003ef91015/ide/image/src/org/netbeans/modules/image/navigation/ImageNavigatorPanel.java#L137-L160 The problem is, that `setNewContent` and `getComponent` race. Normally the former wins and initial view does not happen, because you go out of the `null` check: https://github.com/apache/netbeans/blob/2e720957f1554063363999ff5d938f1a0e556193/ide/svg/src/org/netbeans/modules/svg/navigation/SvgNavigatorPanel.java#L145-L147 I noticed this only after I removed my breakpoints out of the raced path and only into the "it is broken" regions. For the actions: I noticed that too in other contexts. You might want to have a look at the implementation of the "History" multiview element, as that is in a similar role, than preview (not an editor, but should do many thing similar to the companion multiview element). -- 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
