[Asset graph] Add animations for in progress / queued materialization state #20556
Conversation
|
Deploy preview for dagit-storybook ready! ✅ Preview Built with commit 2632512. |
|
Deploy preview for dagit-core-storybook ready! ✅ Preview Built with commit 2632512. |
| // Check if the input string is in the correct format | ||
| const rgbaRegex = /^rgba?\((\s*\d+\s*),(\s*\d+\s*),(\s*\d+\s*),(\s*[\d.]+\s*)\)$/; | ||
| const match = rgbaString.match(rgbaRegex); | ||
| if (!match) { |
There was a problem hiding this comment.
Would be nice to also require match.length === 3 or 4 here, just in case since it's required on lines 21-23
There was a problem hiding this comment.
the regex requires that by the number of matches in it
… state (#20556) ## Summary & Motivation https://github.com/dagster-io/dagster/assets/2286579/eaa4be22-a8ac-4155-be4b-606bf0164593 # Test plan: Storybook: https://dagit-core-storybook-78f45n3pp-elementl.vercel.app/?path=/story/asset-graph-assetnode--live-states
… state (#20556) ## Summary & Motivation https://github.com/dagster-io/dagster/assets/2286579/eaa4be22-a8ac-4155-be4b-606bf0164593 # Test plan: Storybook: https://dagit-core-storybook-78f45n3pp-elementl.vercel.app/?path=/story/asset-graph-assetnode--live-states
Summary & Motivation
Screen.Recording.2024-03-19.at.9.50.38.AM.mov
Test plan:
Storybook: https://dagit-core-storybook-78f45n3pp-elementl.vercel.app/?path=/story/asset-graph-assetnode--live-states