lukaszlenart opened a new pull request, #118:
URL: https://github.com/apache/struts-intellij-plugin/pull/118

   ## Summary
   - Restore the struts.xml Diagram tab on 262 by calling 
`super.selectNotify()`/`deselectNotify()` so `PerspectiveFileEditor` installs 
the custom component into the editor wrapper
   - Always apply the built model on the UI thread (keep `myDiagramSelected` 
only for DomEvent live-refresh scheduling) and coalesce concurrent rebuilds
   - Size EMPTY/UNAVAILABLE panels defensively; add regression tests for model 
apply without selectNotify and wrapper installation after selectNotify
   - Document design/plan; Diagrams API migration remains future work 
([#117](https://github.com/apache/struts-intellij-plugin/issues/117))
   
   ## Test plan
   - [x] `./gradlew test -x rat --tests "com.intellij.struts2.diagram.*"` (30 
tests)
   - [ ] `./gradlew runIde` — open file-set `struts.xml` → Diagram shows 
packages/actions/results (not blank)
   - [ ] Text → Diagram after edit → catch-up refresh still works
   - [ ] Stay on Diagram, edit XML → debounced refresh still works
   - [ ] Empty/unavailable case shows placeholder text, not a blank pane
   
   
   Made with [Cursor](https://cursor.com)


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