iets3.opensource icon indicating copy to clipboard operation
iets3.opensource copied to clipboard

Feature/improve value inspector

Open markusvoelter opened this issue 2 years ago • 5 comments

Some syntactic improvements in value inspectors

markusvoelter avatar Apr 04 '22 08:04 markusvoelter

Looks good to me, just one question: why did you remove and readd the InlineAll intention under inlineAll2 name?

wsafonov avatar Apr 05 '22 09:04 wsafonov

Because there was a problem with this intention - so it looked. In my desparation, I reimplemented it. But the problem was a different one.

markusvoelter avatar Apr 11 '22 10:04 markusvoelter

Because there was a problem with this intention - so it looked. In my desparation, I reimplemented it. But the problem was a different one.

Does it make sense for me to have a look what could be that problem? Because the new intention looks exactly the same as the old one, except for child filter. If not, we can then at least give it the old name, cause having only single one inlineAll2 intention looks strange for later reviews.

wsafonov avatar Apr 11 '22 13:04 wsafonov

No. It was a version incompatibility.

markusvoelter avatar Apr 13 '22 17:04 markusvoelter

If you care about the "2" in the name, you can remove it.

markusvoelter avatar Apr 13 '22 17:04 markusvoelter