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

UX revisions to drill down header #4697

Merged
merged 7 commits into from
Sep 9, 2024
Merged

UX revisions to drill down header #4697

merged 7 commits into from
Sep 9, 2024

Conversation

ngraham76
Copy link
Contributor

Description

  • Removed the kebab menu icon button next to the output dropdown as we no longer need the Save function here - it's now been added to operator's output header as a Save for re-use button
  • Added a 1px bottom border (surface-border-alt) to the operator drilldown headers - defined in our Figma designs, was missing

image

Copy link
Member

@mwdchang mwdchang left a comment

Choose a reason for hiding this comment

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

We need to clean up the callers so they no longer generate and send in "menu-items" unnecessarily.

@mwdchang
Copy link
Member

mwdchang commented Sep 9, 2024

TA3 operators likely needs a bit of design tweaks for removing these menu-item paradigms, as they are saving different types of asset or assets. See follow up in:

#4713

@mwdchang mwdchang merged commit 0509cc1 into main Sep 9, 2024
5 checks passed
@mwdchang mwdchang deleted the neil_2 branch September 9, 2024 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants