Simplify resizable API: use persist: false | 'localStorage' | fn #7395
+91
−60
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Consolidates
resizableprop configuration from separate{persist: false}and{save: fn}patterns into unified{persist: false | 'localStorage' | fn}API.Changelog
Changed
resizableprop now accepts{persist: false | 'localStorage' | fn}instead of separate{persist: false}and{save: fn}configsCustomPersistConfig→PersistConfigNoPersistConfig→ removed (merged intoPersistConfig)persistproperty instead ofsaveBefore:
After:
New
PersistFunctiontype for custom persistenceisCustomPersistFunctiontype guardisPersistConfigtype guardRemoved
NoPersistConfigtype (merged intoPersistConfig)CustomPersistConfigtype (replaced byPersistConfig)isNoPersistConfigtype guardisCustomPersistConfigtype guardRollout strategy
Note: While this changes the API surface, the original PR hasn't been released yet, so this is effectively a pre-release refinement.
Testing & Reviewing
Merge checklist
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.