instead of manual invocation, re-use the text hint interface for auto-hover invocation
do manual hover the same way, just trigger the new functionality
it is a bit hacky, as the text hint interface isn't really made for async stuff,
but it works well ;=)
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
Ah, other question: Would it be ok to do the code stuff under MIT license?
That way, perhaps we can later split out some library things for 3rdparty projects (not that I think it is very likely).
Or would that be some issue?
Looks nice indeed!
I had also been considering another (more liberal) license and MIT is fine by me (not sure whether anyone else needs to agree with re-licensing at this stage ...?)
KDevelop does the same: they always return an empty QString to cancel the texthint stuff and then show their own just like you do now.
They have some more sophisticated tooltip that may be of interest in case the tooltip is not nice/good enough over time.
The KDevelop tooltip is nice and I think copying features from it might be very cool, but I think I wouldn't copy the implementation ;)
Ok, given this feature seems to be appreciated and the implementation ok, I will merge this.
I will change the license header of the new files to MIT, there is no non-MIT code inside, given me and Mark are the only authors.
For re-licensing the other parts:
For some of the other files, I copied over some "boilerplate" to get going, so nothing unique in there and (almost surely) already modified by now into other ways and forms ...
Then feel free to cleanup the copyright of all your files, now is the time when we still know who really contributed where.
Later this will no longer work.
See the copyright headers in the hover stuff, MIT with a proper SPDX identifier for license tooling.