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

Fix margin of expanded alerts #2117

Merged
merged 1 commit into from
Sep 7, 2017
Merged

Conversation

moolitayer
Copy link

@moolitayer moolitayer commented Sep 6, 2017

The provider of the alert ('EngLab Prometheus' here) is too close to the upper margin of the expanded content

before:
before

after:
after

@miq-bot
Copy link
Member

miq-bot commented Sep 6, 2017

Checked commit moolitayer@70d85a6 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks fine. 🍰

@epwinchell
Copy link
Contributor

@moolitayer Looks good

@epwinchell
Copy link
Contributor

@miq-bot add_label fine/no, monitor, formatting/styling

@mzazrivec mzazrivec self-assigned this Sep 7, 2017
@mzazrivec mzazrivec added this to the Sprint 69 Ending Sep 18, 2017 milestone Sep 7, 2017
@mzazrivec mzazrivec merged commit 2900a45 into ManageIQ:master Sep 7, 2017
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.

4 participants