thanosp/phpstorm-inheritdoc

Multi-line virtual text

Opened this issue · 10 comments

thanosp commented

Right now the plugin is using folding to show the inherited text.
However folding does not allow for multiple lines which means the whole text has to be collapsed into one.

  • Find another solution that allows multiline text via intellij
MacDada commented

+1

This is the reason I uninstalled the plugin.

  • 👍1
temp commented

+1
I like the idea of the plugin, but with the one-liner it is really unusable...

nesl247 commented

👍 from me as well. It's still better than nothing, but it does suck to not have multi-line.

Isinlor commented

It is a nice plugin, but indeed it would be more usable in practice with multi-line support. 👍

thanosp commented

If anyone has any idea or seen a similar feature in another plugin I am more than willing to investigate further.
Last time I checked intellij does not provide any other means of virtual multiline text.
The plugin is currently using the folding feature for replacing the inheritdoc tag with virtual text which is constrained to one row. That is also what's used by the symfony plugin for example for mapping route names to route urls.

I will take another look in case the constraint is gone in the latest version but that would basically mean folding could be used to expand text instead and then the folding signs -, + (which is the actual feature of intellij the plugin is abusing) would lose their meaning. I think that is highly unlikely to happen.

mente commented

+1 if it' still maintained

callistino commented

I love this plugin and this issue let me to a good enough solution for me. If you enable the options Show quick documentation on mouse move. You can hover over the method signature and see the docs which include the original phpdoc.

  • 👍1
develpr commented

Just wanted to throw a +1 here - multi-line support is essentially critical, but I do understand that it simply may not be possible!

davyrolink commented

I think Navigating to Declaration or Type Declaration of a Symbol would be much better than the current single-line option. Would that be difficult to implement?

thanosp commented

@davyrolink i would not consider clicks/shortcuts fixing the issue but you can already do that (cmd + U to jump to the super method for osx).

I've opened a ticket also to hopefully get some feedback from intellij devs: https://youtrack.jetbrains.com/issue/IDEA-166134

I am still using it personally even with the one-liner (not the inspections though, that was a bad idea)

  • 👍1