lukaszlenart commented on issue #121:
URL: 
https://github.com/apache/struts-intellij-plugin/issues/121#issuecomment-5080245767

   Fixed on `feat/diagrams-api-migration` / #119.
   
   **Root cause:** In non-popup diagram editor mode, 
`DiagramPresentationModelImpl.editNode` only navigates via 
`DiagramExtras.getEditNodeHandler()`. Overriding `DiagramNode.navigate()` is 
not enough (that path is effectively for popup / focus).
   
   **Fix:** `StrutsDiagramExtras` provides an `EditNodeHandler` that calls 
`StrutsDiagramPresentation.navigateToElement`, plus `uiDataSnapshot` / 
`getItemDocOwner` for Jump to Source.
   
   Please re-check double-click in Show Diagram after pulling the latest PR 
commits.


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

Reply via email to