eui icon indicating copy to clipboard operation
eui copied to clipboard

[Emotion] Convert `EuiFlyout`

Open breehall opened this issue 3 years ago • 3 comments
trafficstars

Summary

This PR converts EuiFlyout to Emotion. There are no functional or UI changes within this conversion.

Created EuiFlyout context as a way to pass the value of the padding prop from EuiFlyout down to EuiFlyoutHeader, EuiFlyoutBody, and EuiFlyoutFooter.

Checklist

  • [x] Checked in both light and dark modes
  • [x] Checked in mobile
  • [x] Checked in Chrome, Safari, Edge, and Firefox
  • ~[ ] Props have proper autodocs and playground toggles~
  • ~[ ] Added documentation~
  • [ ] Checked Code Sandbox works for any docs examples
  • [x] Added or updated jest and cypress tests
  • [x] Checked for breaking changes and labeled appropriately
  • [x] Checked for accessibility including keyboard-only and screenreader modes
  • ~[ ] Updated the Figma library counterpart~
  • [x] A changelog entry exists and is marked appropriately

breehall avatar Aug 04 '22 14:08 breehall

Preview documentation changes for this PR: https://eui.elastic.co/pr_6108/

kibanamachine avatar Aug 04 '22 15:08 kibanamachine

Preview documentation changes for this PR: https://eui.elastic.co/pr_6108/

kibanamachine avatar Aug 04 '22 15:08 kibanamachine

Preview documentation changes for this PR: https://eui.elastic.co/pr_6108/

kibanamachine avatar Aug 08 '22 21:08 kibanamachine

Preview documentation changes for this PR: https://eui.elastic.co/pr_6108/

kibanamachine avatar Aug 30 '22 21:08 kibanamachine

Preview documentation changes for this PR: https://eui.elastic.co/pr_6108/

kibanamachine avatar Aug 31 '22 14:08 kibanamachine

Due to the nature of the changes, I thought it would be best to bring the actual changes over to a new branch and leave everything else to reduce the diff churn from moving around files. I will be closing this PR and using #6213 moving forward. That PR has a summary and a link to the most recent changes requested.

breehall avatar Sep 07 '22 13:09 breehall