Skip to content

Colour the error, info and warning lines in the Debug Window#2275

Open
zedwick wants to merge 14 commits into
overte-org:masterfrom
zedwick:debug_improvements
Open

Colour the error, info and warning lines in the Debug Window#2275
zedwick wants to merge 14 commits into
overte-org:masterfrom
zedwick:debug_improvements

Conversation

@zedwick

@zedwick zedwick commented May 18, 2026

Copy link
Copy Markdown
Collaborator

The Debug Window shows all script log lines in monocolour:

Screenshot_20260519_000642

Which makes it difficult to spot error messages, so I added colours:

Screenshot_20260519_090450

In this PR it also shows a little message before the first log lines show up:

Screenshot_20260519_001427

Users can also no longer edit the log lines shown in the Debug Window, but can select and copy the text.

Note: I updated QtQuick and QtQuick.Controls to get some newer features, which lost a lot of implicit styling from QtQuick.controls 1.x. So I have restyled using HifiConstants.colors from stylesUit. The background is a darker gray, and I had to add the scrollbar back in; it is a little different.

zedwick added 3 commits May 19, 2026 00:30
I updated QtQuick and QtQuick.Controls to get some newer features, which lost a lot of implicit styling from QtQuick.controls 1.x. So I have restyled using HifiConstants.colors from stylesUit.
The background is a darker gray, and the scrollbar is a little different.
@zedwick zedwick added enhancement New feature or request needs CR This pull request needs to be code reviewed needs QA This pull request needs to be tested labels May 19, 2026
Comment thread scripts/developer/debugging/debugWindow.js Outdated
Comment thread scripts/developer/debugging/debugWindow.qml Outdated
zedwick and others added 2 commits May 21, 2026 13:19
Co-authored-by: Ada <ada+github@thingvellir.net>
@zedwick zedwick requested a review from ada-tv May 21, 2026 13:22
@ada-tv ada-tv added CR approved This pull request has been successfully code reviewed and removed needs CR This pull request needs to be code reviewed labels May 21, 2026
@zedwick zedwick marked this pull request as draft May 21, 2026 14:34
@zedwick zedwick marked this pull request as ready for review May 22, 2026 15:47
@zedwick zedwick requested a review from ada-tv May 22, 2026 15:47
@zedwick

zedwick commented May 22, 2026

Copy link
Copy Markdown
Collaborator Author

I had a bit of trouble with scrolling, but I think I have it sorted properly now so that removing log lines no longer breaks the built-in automatic scrolling. I kept in some manual scrolling just in-case, but we do it less often than before.

@zedwick zedwick added needs CR This pull request needs to be code reviewed and removed CR approved This pull request has been successfully code reviewed labels May 22, 2026
@zedwick

zedwick commented Jun 6, 2026

Copy link
Copy Markdown
Collaborator Author

I have now also added filtering to the Debug Window.

Screenshot_20260606_212851

Can now select a script name from the drop down list at the bottom of the window, which will clear the log lines from the text area above and then only show log lines associated with the selected script name.

Script names are added to the drop down as its log messages are received, so the list will start out empty and only allow filtering once the first log line from that particular script has been received by the window.

Comment on lines +110 to +115
Text {
text: "-->"
color: hifi.colors.lightGrayText
horizontalAlignment: Text.AlignHCenter
clip:true
}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A label next to the control is good, the arrow isn't necessary.

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

Labels

enhancement New feature or request needs CR This pull request needs to be code reviewed needs QA This pull request needs to be tested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants