Error message hints displayed weirdly if there is also a source context

[msullivan]

asdf> select sequence_next(100);
error: QueryError: function "sequence_next(arg0: std::int64)" does not exist
  ┌─ query:1:8
  │
1 │ select sequence_next(100);
  │        ^^^^^^^^^^^^^^^^^^ Did you want "std::sequence_next(seq: schema::ScalarType)"?

The hint appears on the same line as the carets indicating where the error was, and in red.

[tailhook]

Yes this is by design. I'm not sure we have a good policy of what goes to "hint" and goes to "detail", though. I think we should review that when refactoring errors, since pointer that always says "error" is not great too.