thsmi/sieve

[annoyance][standalone app] The error message covers the editor

Closed this issue · 2 comments

Prerequisites

  • Tried the most recent nightly build (where are nightlies?)
  • Checked if your issue is already reported.
  • Answered all the questions in this template (Or provide a working crystal ball).

What happened?

When editing a script with a syntax error, an error message appears at the bottom of the editor, which partially covers the script itself. If the cursor happens to be at the bottom, you can then no longer see what you type.

  1. Connect to some server and edit a script.
  2. Introduce any syntactic error.
  3. Error message appears over the editor.

What did you expect to happen?

The error message not covering any important UI elements. Or maybe only show a hint (red underline?) where the error is and only show the message after a click.

Screenshots

image

Which Version

  • Standalone app

The error message is returned by the server and completely transparent for a client. Each server returns a different message style and format. Thus there is no way to parse it and relate the error message to a certain line.

And honestly I don't get what you mean with covering important UI elements. The error message floats like the toolbar on the top. The errors are only shown when needed to preserve valuable screen real estate. Because it floats it will never overlay an active line. Instead the the editor will scroll the active line into view.

So see lines outside of you view just use the scroll bar. If you scroll down for example in your screenshot the toolbar will overlay the top of your code, but this is normal and what scrollbars or other floating elements are intended to do.

If the error message is displayed you'll still be able to scroll the the editors very bottom and see everything without the error message overlaying something.

In case the error message would be always displayed with a fixed height on the bottom regardless if it is needed or not then it would eat up you valuable screen space just for nothing.

If you don't want to have syntax checking at all. Then just disable it in the account settings and the message bar will be gone.

Information needs to go somewhere space on a screen is limited and I don't se any other way to display this information. So I'll close this as a won't fix.

Thank you for your feedback.

Information needs to go somewhere space on a screen is limited and I don't se any other way to display this information. So I'll close this as a won't fix.

I agree but one place it cannot go is over the cursor, as it does right now. If the line number cannot be parsed reliably because of the different formats, what about a button in the toolbar which appears if an error is detected? The error message could be accessed by clicking the button.