getappmap / appmap-intellij-plugin

MIT License

38 stars 9 forks source link

Add Navie section to the AppMap toolwindow #672

Closed jansorg closed 7 months ago

jansorg commented 7 months ago

Closes https://github.com/getappmap/appmap-intellij-plugin/issues/668

Questions:

In a project without AppMaps, a click on "Open instructions" opens the installation guide. Is this still correct?

I've tested this with Light and Dark themes of 2022.1 and briefly with 2024.1.

All panels expanded:

Default state:

Project without AppMaps:

github-actions[bot] commented 7 months ago

AppMap runtime code review

Summary	Status
Failed tests	:white_check_mark: All tests passed
API changes	:zero: No API changes
Security flaws	:white_check_mark: None detected
Performance problems	:white_check_mark: None detected
Code anti-patterns	:white_check_mark: None detected
New AppMaps	:zero: No new AppMaps

jansorg commented 7 months ago

@ahtrotta I've applied the requested changes except for the button. I've also updated the Navie webvomponent because it was still displaying the red warning about missing AppMaps. Implementing the button isn't straight-forward, unfortunately. There's not built-in support for it and I'll have to implement a custom Swing component to make this work. I was planning to implement #667 before adding the button.

appland-release commented 7 months ago

:tada: This PR is included in version 0.65.0 :tada:

The release is available on:

v0.65.0
GitHub release

Your semantic-release bot :package::rocket: