ENG-11392: Update gradient profile shared component - #6913
Conversation
Merging this PR will not alter performance
Comparing Footnotes
|
Greptile SummaryThe PR replaces the external gradient-profile dependency with a packaged shared JavaScript component and resolves that asset lazily when the Python component is created.
Confidence Score: 5/5The PR appears safe to merge because no blocking failure remains. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| packages/reflex-components-internal/src/reflex_components_internal/components/base/GradientProfile.js | Adds the local memoized React component that deterministically derives and renders a two-color gradient. |
| packages/reflex-components-internal/src/reflex_components_internal/components/base/gradient_profile.py | Migrates GradientProfile to a lazily resolved shared asset while preserving caller-supplied library overrides. |
| tests/units/reflex_components_internal/components/base/test_gradient_profile.py | Adds coverage for lazy shared-asset resolution, library overrides, style composition, and decorative semantics. |
Reviews (3): Last reviewed commit: "update" | Re-trigger Greptile
There was a problem hiding this comment.
All reported issues were addressed across 2 files
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
There was a problem hiding this comment.
All reported issues were addressed across 3 files (changes from recent commits).
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
Uh oh!
There was an error while loading. Please reload this page.