> What about a hybrid where if there's no parameter list and no scope info then > show file and line.
Nobody will understand that. For users it will just confusing why they sometimes see filenames and sometimes the signature. And as for me (and apparently @kugel- and @elextr too), always seeing file names is just necessary. > @techee how are you getting those text declarations? It seems like there > should be some scope info probably. Frankly, I did't properly updated your branch on my machine, they are present now, sorry. But the goto popup has to work reasonably well for all languages, such as Python where you don't have the type information or some other parsers that don't support scope parsing (or there's no concept of scope in the languages themselves). > BTW this pull only adds info to the menu so I'd like to merge it. No, it doesn't - this PR _removes_ the file name from the popup which simply isn't good and would have to be fixed by any subsequent PR. Also, modifying `tm_parser_format_variable()` means the scope will also be shown in the tooltips of the symbols in the sidebar which IMO adds unnecessary clutter as this info is visible from the symbol tree itself. Rather than this PR, even though @b4n maybe thinks otherwise, it would be better to apply #3495 (just ellipsizing in a smarter way) and avoid the configuration altogether. > I can make it configurable. Would you wait for a few days? I'd like to play with this a bit and see what works and what not. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3542#issuecomment-1688927660 You are receiving this because you are subscribed to this thread. Message ID: <geany/geany/pull/3542/[email protected]>
