Skip to content

Conversation

@Arnav05-cyber
Copy link

  • Changed _OutputWidgetHandler.emit() to append new messages to the end
  • This ensures errors and new log messages appear at the bottom where they're more visible
  • Fixes issue where errors could be hidden when appearing at the top of log window

Fixes: GUI error visibility issue

- Changed _OutputWidgetHandler.emit() to append new messages to the end
- This ensures errors and new log messages appear at the bottom where they're more visible
- Fixes issue where errors could be hidden when appearing at the top of log window

Fixes: GUI error visibility issue
@dylansdaniels
Copy link
Collaborator

Hi, @Arnav05-cyber. Thanks for your PR. We actually decided to have the latest messages displayed at the top because of how the log automatically scrolls to the top. This ensures users actually see the latest messages as they are printed to the log in real time. So this is actually an intended behavior, not an error :)

@dylansdaniels dylansdaniels added wontfix Indicates that work won't continue on an issue, pull request, or discussion blocked/cannot proceed and removed wontfix Indicates that work won't continue on an issue, pull request, or discussion labels Oct 29, 2025
@asoplata
Copy link
Collaborator

asoplata commented Dec 8, 2025

As Dylan said, this is actually the opposite of what we want: it would be better if errors were printed to the top of the log like other log information is, instead of everything being printed at the bottom: #1163 . If we want to change this PR such that it is about printing errors at the top instead of the bottom, then we can keep it open.

@dylansdaniels
Copy link
Collaborator

@asoplata I believe this is how it works already, though. So I don't think any change is needed

@asoplata
Copy link
Collaborator

asoplata commented Dec 8, 2025

True Python errors (such as errors during simulation time) appear to go on the bottom, as far as I've been able to tell. I've listed an example here: #1163 (comment)

@dylansdaniels
Copy link
Collaborator

I see, good catch. We definitely want those printed at the top as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

3 participants