eui icon indicating copy to clipboard operation
eui copied to clipboard

[Emotion][perf] Memoize low-impact components (D-F)

Open cee-chen opened this issue 1 year ago • 3 comments

Summary

Basically every component that isn't on the medium impact list (https://github.com/elastic/eui/issues/7561) and was easier to do a quick skim through. Some of the commits/components have extra cleanup in terms of useMemo usage, preferring inline JSX, and removing array/obj definitions. As always, I recommend following by commit.

QA

  • [ ] Quick smoke test of all the affected components

General checklist

  • Browser QA - Skipping cross-browser testing in favor of smoke testing
  • Docs site QA - N/A
  • Code quality checklist - N/A
  • Release checklist - N/A, skipping changelog due to this being perf tech debt that shouldn't impact consumers
  • Designer checklist - N/A

cee-chen avatar Mar 29 '24 21:03 cee-chen

Preview staging links for this PR:

  • Docs site: https://eui.elastic.co/pr_7637/
  • Storybook: https://eui.elastic.co/pr_7637/storybook

kibanamachine avatar Mar 29 '24 23:03 kibanamachine

:green_heart: Build Succeeded

History

  • :green_heart: Build #1634 succeeded 619e1a80847a1a6e1e9f6e7e776d9f4e7538b1bc

elasticmachine avatar Mar 29 '24 23:03 elasticmachine

👋 Hey there. This PR hasn't had any activity for 90 days. We'll automatically close it if that trend continues for another week. If you feel this issue is still valid and needs attention please let us know with a comment.

github-actions[bot] avatar Jun 28 '24 00:06 github-actions[bot]