Skip to content

Conversation

joshistoast
Copy link
Contributor

Summary

Made some UI updates to the workflow library modal in order to make it feel a bit better.

  1. Improved interactions on the side nav.
  2. Tweaked UI of list items
  3. Darkened background of modal
  4. Changed "View" to "Load" since that's what the button does.
  5. Moved some styling to system style objects rather than inline attributes.

Just a heads up, I haven't tested this on the workflow sharing and teams oriented stuff, so I could use some feedback on that if the UI there looks odd.

image

Related Issues / Discussions

QA Instructions

Merge Plan

Checklist

  • The PR has a short but descriptive title, suitable for a changelog
  • Tests added / updated (if applicable)
  • ❗Changes to a redux slice have a corresponding migration
  • Documentation added / updated (if applicable)
  • Updated What's New copy (if doing a release after this PR)

@github-actions github-actions bot added the frontend PRs that change frontend files label Sep 17, 2025
Copy link
Collaborator

@maryhipp maryhipp left a comment

Choose a reason for hiding this comment

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

Looks good! I tested with team features on and left one comment suggesting that the blue line indicating active state would look nicer flush on the side as more of a border

left: 1,
w: 0.5,
rounded: 'sm',
bg: selectedView === view ? 'invokeBlue.300' : 'transparent',
Copy link
Collaborator

Choose a reason for hiding this comment

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

Rather than using a pseudo-element here, how about a border? Regardless I would have the blue active state be full height and flush. Here are some screenshot of the different states - the smaller elements when team features are enabled looks better with the blue line flush IMO

Screenshot 2025-09-17 at 9 30 31 AM Screenshot 2025-09-17 at 9 30 20 AM

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Did it this way to be more consistent with the model manager active state, also using a full height border would taint the border radius, no?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend PRs that change frontend files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants