Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(ui): Playground settings / show more / trials area #3311

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

m-rgba
Copy link
Contributor

@m-rgba m-rgba commented Dec 24, 2024

Description

  • Updated trials view to be re-sizable (added re-sizable component which can be used for alternative drawers).
  • Tweaked trial header buttons / added sticky header.
  • Updated Show More buttons to be smaller and fixed old overlay issues to have them sit on top (updated the backgrounds to make them a little less visually weighted as well).
  • Fixed issue with MessagePanel and tool calls having show more + max-height.

Screenshots

Description Screenshot
Updated trials drawer + floating headers Screenshot 2024-12-30 at 10 12 45 AM
Smaller show more buttons and re-added overlay fade Screenshot 2024-12-30 at 10 13 10 AM
Fixed issue with show more buttons and max height for tools Screenshot 2024-12-30 at 10 13 34 AM

@circle-job-mirror
Copy link

@m-rgba
Copy link
Contributor Author

m-rgba commented Dec 30, 2024

@m-rgba m-rgba marked this pull request as ready for review December 30, 2024 15:54
@m-rgba m-rgba requested review from a team as code owners December 30, 2024 15:54
Copy link
Contributor

@jwlee64 jwlee64 left a comment

Choose a reason for hiding this comment

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

Lgtm, except a small fix necessary

Screen.Recording.2025-01-02.at.8.11.37.AM.mov

I think the show more is resizing in the drawer incorrectly.
The show more is only showing the last like 20 pixels regardless of the height of the content.

@m-rgba
Copy link
Contributor Author

m-rgba commented Jan 2, 2025

Lgtm, except a small fix necessary

Screen.Recording.2025-01-02.at.8.11.37.AM.mov
I think the show more is resizing in the drawer incorrectly. The show more is only showing the last like 20 pixels regardless of the height of the content.

Will fix - interesting, yeah it only looks like that's in the trials, in the regular chat views it seems to be the right height
Example via: https://beta.wandb.ai/wandb-designers/winston/weave/playground/0193f98a-540c-71f2-a794-d24dcab2af3c
image
image

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