esteinberg / plantuml4idea

Intellij IDEA plugin for PlantUML
Apache License 2.0
621 stars 112 forks source link

Automatic reload/rendering can cause problems with other files when in split view. #344

Closed Beliar83 closed 2 years ago

Beliar83 commented 2 years ago

I have noticed that if I move the plant uml file into a separate view and change code in the other view while the plant uml file is open that the IDE performs "on save" actions, like formatting, on the file I am typing in. This can cause irritation when the cursor ends up at an unexpected position.

Can you see if you can limit the rerendering of diagrams to only happen when it is a plant uml file that is being changed?

krasa commented 2 years ago

cannot reproduce, for me a file is rendered after its change or change of its includes, not change of some unrelated file please record it.

Beliar83 commented 2 years ago

Here, it should show, that having it open seems to influence other files to some degree. I am using Version 5.10.0, btw, which is the latest available according to Clion.

https://user-images.githubusercontent.com/728747/152499749-527a6ef1-1a02-4417-b132-d8390e0ee9c9.mp4

krasa commented 2 years ago

not even VLC can play it, if you are on Win then https://www.screentogif.com/ is good

Beliar83 commented 2 years ago

Sorry, and thanks for that link: Animation

krasa commented 2 years ago

Strange, I would understand that editing puml reformats your code via save-actions-plugin, but editing other files should not trigger rendering (that saves all files). I will have to reproduce it. Or perhaps you could enable debug logging for #org.plantuml and post the log.

krasa commented 2 years ago

Or it could be the Annotator, try to disable syntax checking. image

Yes, that must be it, I will fix it.

Beliar83 commented 2 years ago

Yeah, that option seems to be the problem.

krasa commented 2 years ago

You can try the fix (install the whole zip): plantuml4idea.zip

Beliar83 commented 2 years ago

Okay, had to solve a problem with clion, but the problem with this plugin seems to be fixed now. Thanks