[Uptime] Update ML flyout to add cancel button #61627
Conversation
shahzad31
added
release_note:skip
|
Pinging @elastic/uptime (Team:uptime) |
| <EuiSpacer /> | ||
| </EuiFlyoutBody> | ||
| <EuiFlyoutFooter> | ||
| <EuiFlexGroup justifyContent="spaceBetween" alignItems="flexEnd"> | ||
| <EuiFlexItem grow={true}> |
There was a problem hiding this comment.
| <EuiFlexItem grow={true}> | |
| <EuiFlexItem grow={false}> |
There was a problem hiding this comment.
if i set this to false, create new button will take all the space.
There was a problem hiding this comment.
i need to remove alignItems="flexEnd"
| <EuiSpacer /> | ||
| </EuiFlyoutBody> | ||
| <EuiFlyoutFooter> | ||
| <EuiFlexGroup justifyContent="spaceBetween" alignItems="flexEnd"> | ||
| <EuiFlexItem grow={true}> | ||
| <EuiButtonEmpty | ||
| style={{ width: 100 }} |
There was a problem hiding this comment.
I'm not sure - why do we need to set the width?
There was a problem hiding this comment.
it was taking half width and hence was pusing cancel button to center, i think maybe i could have achieved this by using justifyContent to flexStart
There was a problem hiding this comment.
let me check an EUI example to achieve this best using EUI
|
@katrin-freihofner updated the code. Removed width. |
shahzad31
force-pushed
the
update-anomaly-detection-flyout
branch
from
8a2e662 to
c9c2d54
Compare
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
* update button * fix type * update snaps * update layout
* update button * fix type * update snaps * update layout
Summary
Fixes: #61584
Added a cancel button on ML flyout