Skip to content
This repository has been archived by the owner on Jan 8, 2024. It is now read-only.

Project settings polish #1335

Merged
merged 5 commits into from
Apr 16, 2021
Merged

Project settings polish #1335

merged 5 commits into from
Apr 16, 2021

Conversation

dizzyup
Copy link
Contributor

@dizzyup dizzyup commented Apr 14, 2021

Wanted to do a little work on the form layout/design for project settings.

Also fixes a bug I noticed where we were mistakenly removing the bottom border on buttons, and a regression where we lost our tab styles in some of the onboarding steps.

New project page

CleanShot 2021-04-14 at 18 24 56@2x

Project settings page

CleanShot 2021-04-14 at 18 24 31@2x

@dizzyup dizzyup self-assigned this Apr 14, 2021
@github-actions github-actions bot added the ui label Apr 14, 2021
@mitchellh
Copy link
Contributor

mitchellh commented Apr 14, 2021

Let's also backport this puppy when it becomes time. 🐶 (Just label it backport/0.3.x)

Copy link
Contributor

@gregone gregone left a comment

Choose a reason for hiding this comment

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

✨ looks great with the card sections! You'll need to add a changelog entry before merging

@dizzyup dizzyup marked this pull request as ready for review April 15, 2021 11:36
@dizzyup dizzyup merged commit 94a6c04 into main Apr 16, 2021
@dizzyup dizzyup deleted the settings-polish branch April 16, 2021 12:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants