Godot: Long error messages are displayed outside screen in console

Created on 1 Jan 2018  路  5Comments  路  Source: godotengine/godot

Godot version:

Built from f11a1385054376bdc4cc01e6aa20d63508a5e6d4 with PR #15136.

OS/device including version:

_Majaro Linux 17.1-rc2_

Issue description:

When a long error message is logged to the console, it is impossible to read the whole content since the displayed tooltip is bigger than the screen size, as shown below:

godot-console

Ideally, console should show the detailed content in a separate view, when the user selects a message from the list. It is especially important when the message contains a stacktrace information.

Steps to reproduce:
Write a code that throws a runtime exception. I tried it with C# project, so I'm not too sure if the problem persists with a GDScript project.

  • [x] I searched the existing GitHub issues for potential duplicates.
bug editor usability

Most helpful comment

How does this look for a quick mock-up?

err_details_mockup

I used a VSplitContainer on the left and a read-only TextEdit for the details output, which shows the text normally found only in the tooltip. It allows selection for copying, as shown in the picture.

Alternatively, we could put the details split on the right side to give more height priority to the Error list, if the Stack Trace list doesn't require as much height.

All 5 comments

How does this look for a quick mock-up?

err_details_mockup

I used a VSplitContainer on the left and a read-only TextEdit for the details output, which shows the text normally found only in the tooltip. It allows selection for copying, as shown in the picture.

Alternatively, we could put the details split on the right side to give more height priority to the Error list, if the Stack Trace list doesn't require as much height.

@RyanStein Looks fantastic to me :) I think it makes sense to put the stack trace pane to the right side, since stack information tends to grow vertically.

The only improvement I can think of is making details and stack trace panes collapsible, but it's a minor one and I'd be more than happy to see it implemented as it is.

@mysticfall Thank you for the feedback. I've submitted the PR. The panes are not collapsible, though they _are_ contained in SplitContainers, which allows them to be resized as needed, even very minimally so.

Can't wait to try it out. Thanks much for the work! 馃槃

A cool PR was proposed to fix this in #15272, but couldn't be merged eventually due to merge conflicts and the PR getting stale. It's up for a rebase if any contributor is interested.

Was this page helpful?
0 / 5 - 0 ratings