Skip to content

various dashboard fixes for Issues.#217

Merged
mikegros merged 3 commits intolanl:mainfrom
ndebard:ndebard/dashboard-fixes
Mar 28, 2026
Merged

various dashboard fixes for Issues.#217
mikegros merged 3 commits intolanl:mainfrom
ndebard:ndebard/dashboard-fixes

Conversation

@ndebard
Copy link
Copy Markdown
Collaborator

@ndebard ndebard commented Mar 28, 2026

OK this addresses:

It also changes:

  • Settings so it's visible even on smaller screens
  • and adds settings "panes" (like Mac) to help organize settings
  • and adds a settings for "UI" - that has scrollback # of lines, which has a large # of.

Example of running - visible, colorized, pulsating green dot (see screenshot)
image

Example of settings page (screenshot) - notice (1) the sub-pages on the left hand side, and (2) the UI settings pane now.
image

@ndebard
Copy link
Copy Markdown
Collaborator Author

ndebard commented Mar 28, 2026

@mikegros - you're teh dashboard guy. can you take a look at this?

@ndebard
Copy link
Copy Markdown
Collaborator Author

ndebard commented Mar 28, 2026

Oh. Also I fixed the animations from Rich console screwing up (not animating) in the stdout window. Most apparent during the planning spinning.

@mikegros
Copy link
Copy Markdown
Collaborator

This looks awesome I'll try to merge it soon.

@ndebard
Copy link
Copy Markdown
Collaborator Author

ndebard commented Mar 28, 2026

That would be awesome if you thought it was ok to merge. My intention is to use this for an upcoming demo as well as some of our users who are using the dashboard - id like to update their versions.

@mikegros
Copy link
Copy Markdown
Collaborator

This looks sweet and on some very brief tests, looks like an awesome improvement.

I havent thoroughly tested it and don't know that I will have time to today, but a couple small pieces of feedback:

  • Because I dont have time to test all the settings options, if you can vouch for them working as intended, I will trust you there.
  • Its not clear to me what Stdout buffer lines, Events page size, and Max event pages mean from the UI (so I suspect it will not be clear for a user). Is there a better set of names for these parameters?

If you can address that I would be happy to merge today. This is really nice.

@ndebard
Copy link
Copy Markdown
Collaborator Author

ndebard commented Mar 28, 2026

I agree, @mikegros - so I removed those page related settings from Settings. I also added in light/dark/follow system theme for Theme like screenshot.
image

@ndebard
Copy link
Copy Markdown
Collaborator Author

ndebard commented Mar 28, 2026

@mikegros - no rush, but I think it's back in your hands to take a look.

@mikegros mikegros merged commit 78086e5 into lanl:main Mar 28, 2026
1 of 2 checks passed
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.

2 participants