In D9484#365385, @cullmann wrote:

I think we should not start to be over-verbose, lets skip this ATM.

In D9484#182791, @dhaumann wrote:

What happens, if you edit several remove documents and switch a lot.
Will this "spam" the current view with the info message again and again?

Thank you very much for the commit, and for the feedback that improved the patch a lot.
Here is the corresponding diff for SlotSync: D9484

In D9244#182322, @cullmann wrote:

For the commit, shall I push?

After reading the doc, and a bit of source code, it does not seem feasible.
But setting a shorter time of 2s, and Immediate autohide
yield a much better behavior.

Done, but running ultimate tests too quickly,
I hit several times F4 before noticing the 'not local file' error.
Then i had to wait for it to disappear, hit somewhere, wait for the error to appear, ...
Would it be possible to catch that the error is still shown (for the same file),
and allow us to do nothing until the error is fixed ?
Or simpler: a way to collapse identical error messages for the same view into one ?
To the minimum, how to allow the user to quickly remove this error message (a delete cross) ?

In D9244#181290, @cullmann wrote:

If you would remove that, you would first need to write code that does the download via KIO and stuff.

That might be awesome.
But a simpler use case is when switching between few remote files,
knowing exactly where they are on the server, so that
a simple script can curl the right one (based on its name), and execute it.

Thanks you very much for the very enlightening feedback.

A non-valid URL is no local file (and I doubt somebody will ever get one, beside "no" url).
But I can live with two different messages, as long as both are inline via KTextEditor::Message, it makes no sense that one is a messagebox.

To me it still makes sense (shorter code, simpler to maintain, for the very unlikely event)
It would have been nice to explain why (documentation ?) you are so confident
that a file without any valid url
has for sure a valid display (for the error message) nonetheless.

In D9244#178699, @cullmann wrote:

Because I thought that if the url itself was invalid, then things were probably really bad, and it was safer not to rely on the visibility of the document ?

I think the best is to skip that and just warn if no local file, we don't care why it is no one ;=)

In D9244#178650, @cullmann wrote:

Intermediate variables like document render the code much cleaner.
Are they OK ?

That is OK ;=)
But I don't understand why the "no valid url" is a message box but the other is a nice overlay message ;=)

The KateConsole::slotRun() function has been reworked,
with separate errors:
KMessageBox::sorry(0, i18n ("Invalid document URL"));
and
KTextEditor::Message(i18n("Not a local file: '%1'", u.path() )...);

In D9244#178027, @cullmann wrote:

For the error message: Two different errors would be nice.

Btw., you could use the ktexteditor/message.h to show the error message inline in the view, that will make it easier to spot.

Thanks for the positive feedback.