-
Notifications
You must be signed in to change notification settings - Fork 841
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
Center align logos #2246
Center align logos #2246
Conversation
8d90edf
to
d3ecf08
Compare
Thank you, @ryankeairns! Sorry I haven't had time to tackle it myself. |
No problem, I just saw the issue this morning and thought it looked like a quick fix! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed snapshots for ya. Checked these worked. Thanks @ryankeairns
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
* center align logos * fix tests
Fixes #2244
Summary
Center align the SIEM, Logging, and ECE logos; remove cruft.
After
Checklist
Checked in IE11 and FirefoxProps have proper autodocsAdded documentation examplesAdded or updated jest testsChecked for breaking changes and labeled appropriatelyChecked for accessibility including keyboard-only and screenreader modes