-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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 save and cancel buttons mis-aligned in settings #21898
Conversation
Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually? |
Thanks for the fix, @aaronoah! The Jest snapshots for this component will need to be updated otherwise CI will fail. You can run this from the root directory with |
00859e1
to
cae2caf
Compare
cae2caf
to
e960337
Compare
@jen-huang Thanks for helping me! I tried to update the snapshots but it didn't seem to have obsolete ones. Could you check it again? I've rewound the head against the master |
@aaronoah My mistake! We have jest tests for the functionality of the action buttons, but they do not render snapshots, so this change does not need snapshot updates 🙂Kicking off CI now, thank you! |
jenkins, test this |
💔 Build Failed |
retest |
💚 Build Succeeded |
Closes #21720.
@bhavyarm