Skip to content

feat: Create a banner for displaying debuggerRequestResults in a unified and styled fashion#387

Merged
codyward-skybox merged 3 commits into
mainfrom
cody.ward/better_message_layout
May 28, 2026
Merged

feat: Create a banner for displaying debuggerRequestResults in a unified and styled fashion#387
codyward-skybox merged 3 commits into
mainfrom
cody.ward/better_message_layout

Conversation

@codyward-skybox
Copy link
Copy Markdown
Contributor

This change makes the result message rendering unified between tabs (currently the Entity Systems profiler and the CPU profiler).

It also adds styling to make the errors/success cases more apparent. It also adds support for multiline messages.

Some examples:
error_bubble
neutral_bubble
success_bubble

@JakeShirley
Copy link
Copy Markdown
Member

Maybe add a small (?) next to the player list control (on all of the panes) that basically says the same thing. "Don't see your player here? Enable this setting..."

Copy link
Copy Markdown
Contributor

@anthonywongskyboxlabs anthonywongskyboxlabs left a comment

Choose a reason for hiding this comment

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

Looks awesome!

Comment thread webview-ui/src/diagnostics_panel/controls/DebuggerRequestResult.tsx
Comment thread webview-ui/src/diagnostics_panel/utilities/debuggerRequestResult.tsx Outdated
Comment thread webview-ui/src/diagnostics_panel/controls/DebuggerRequestResult.tsx
@codyward-skybox
Copy link
Copy Markdown
Contributor Author

Maybe add a small (?) next to the player list control (on all of the panes) that basically says the same thing. "Don't see your player here? Enable this setting..."

Added a tooltip to all the selection dropdowns:
image

@codyward-skybox codyward-skybox merged commit 54ab07c into main May 28, 2026
2 checks passed
@codyward-skybox codyward-skybox deleted the cody.ward/better_message_layout branch May 28, 2026 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants