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

Admin: Fixes close('X') button layout issue in API keys page #27625

Merged
merged 2 commits into from
Sep 17, 2020

Conversation

nikasvan
Copy link
Contributor

What this PR does / why we need it:
Fixes the form layout issue. Close button was breaking the layout.

Which issue(s) this PR fixes:
#27322

@nikasvan nikasvan requested review from a team, hugohaggmark and peterholmberg and removed request for a team September 16, 2020 17:20
@nikasvan nikasvan changed the title ApiKeys: Fixes close('X') button layout issue in add API key form ApiKeys: Fixes close('X') button layout issue Sep 16, 2020
@zoltanbedi zoltanbedi added pr/external This PR is from external contributor type/bug labels Sep 16, 2020
@torkelo
Copy link
Member

torkelo commented Sep 16, 2020

looks like we have a snapshot test, you can update it with `just -u ApiKeys" and checking the updated snapshot!

Thanks for fixing this! ❤️

Copy link
Contributor

@hugohaggmark hugohaggmark 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 for contributing! ⭐

@hugohaggmark hugohaggmark added this to the 7.2.0-beta2 milestone Sep 17, 2020
@torkelo torkelo merged commit 5a06ed4 into grafana:master Sep 17, 2020
hugohaggmark pushed a commit that referenced this pull request Sep 17, 2020
* ApiKeys: Fixes add API key layout

* ApiKeys: snapshot tests updated

(cherry picked from commit 5a06ed4)
hugohaggmark pushed a commit that referenced this pull request Sep 17, 2020
* ApiKeys: Fixes add API key layout

* ApiKeys: snapshot tests updated

(cherry picked from commit 5a06ed4)
@marefr marefr changed the title ApiKeys: Fixes close('X') button layout issue Admin: Fixes close('X') button layout issue in API keys page Sep 17, 2020
ryantxu pushed a commit that referenced this pull request Nov 18, 2020
* ApiKeys: Fixes add API key layout

* ApiKeys: snapshot tests updated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add to changelog pr/external This PR is from external contributor type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants