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

Update bundle CTA UI. #4210

Merged
merged 4 commits into from
Aug 16, 2022
Merged

Update bundle CTA UI. #4210

merged 4 commits into from
Aug 16, 2022

Conversation

leilenah
Copy link
Collaborator

Reasons for making this change

Bundle CTAs are currently aligned at the bottom of the bundle content section. This creates a lot of space in between bundle content and bundle CTAs.

This change does a couple things:

  1. Removes the space between bundle content and bundle CTAs
  2. Updates bundle CTA UI to aesthetically align with the CTAs in the NewRun form

Related issues

#4189

Screenshots

Screen Shot 2022-08-16 at 11 15 33 AM

Checklist

  • I've added a screenshot of the changes, if this is a frontend change
  • I've added and/or updated tests, if this is a backend change
  • I've run the pre-commit.sh script
  • I've updated docs, if needed

Copy link
Collaborator

@percyliang percyliang left a comment

Choose a reason for hiding this comment

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

Great, thanks!

@mergify mergify bot merged commit 58d74fb into master Aug 16, 2022
@mergify mergify bot deleted the feature/4189-config-panel-buttons branch August 16, 2022 19:36
@leilenah leilenah mentioned this pull request Aug 22, 2022
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.

Edit/rerun and kill buttons in bundle panel should be more accessible
2 participants