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

Add styles to labels in scan jobs form #2341

Merged
merged 1 commit into from
Nov 22, 2019

Conversation

SEAjamieD
Copy link
Contributor

@SEAjamieD SEAjamieD commented Nov 22, 2019

🔩 Description: What code changed, and why?

The styling in https://a2-local-inplace-upgrade-dev.cd.chef.co/jobs/add is out of wack in the Add Nodes portion of the form. Specifically elements are laid out inline. Turned out that the styling for the labels was missing. I've replaced the label styling and now they are back to normal.

⛓️ Related Resources

fixes #2240

👍 Definition of Done

Form elements are stacking on top of each other, as they are in acceptance.

👟 How to Build and Test the Change

  1. build components/automate-ui-devproxy && start_all_services
  2. navigate to https://a2-dev.test/jobs/add#add-nodes, see that the form elements match the acceptance image below or here https://a2-local-inplace-upgrade-acceptance.cd.chef.co/jobs/add

✅ Checklist

  • Tests added/updated?
  • Docs added/updated?

📷 Screenshots, if applicable

Dev - broken
dev

Acceptance - correct
acceptance

This Branch
branch

@SEAjamieD SEAjamieD requested review from a team and susanev November 22, 2019 00:13
@SEAjamieD SEAjamieD self-assigned this Nov 22, 2019
@SEAjamieD SEAjamieD added automate-ui layout bugs related to the recent layout change https://github.com/chef/automate/pull/1485 ui bug 🐛 Something isn't working labels Nov 22, 2019
@susanev susanev requested a review from vjeffrey November 22, 2019 14:38
Copy link

@vjeffrey vjeffrey left a comment

Choose a reason for hiding this comment

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

thank you!

@susanev susanev merged commit ee615c7 into master Nov 22, 2019
@chef-expeditor chef-expeditor bot deleted the jamie/2240-scan-jobs-text-styling branch November 22, 2019 18:54
kagarmoe pushed a commit that referenced this pull request Nov 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automate-ui bug 🐛 Something isn't working layout bugs related to the recent layout change https://github.com/chef/automate/pull/1485 ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

scan jobs create page weird text styling
3 participants