Closed benjaminpkane closed 17 hours ago
The changes in this pull request involve modifications to the useSetModalState.ts
and options.ts
files related to the management of the dynamicGroupsViewMode
state. In useSetModalState.ts
, references to dynamicGroupsViewMode
have been removed from the data array. In options.ts
, the dynamicGroupsViewMode
atomFamily has been renamed and its type updated to allow null
values, along with the introduction of a new selectorFamily for enhanced state handling.
File Path | Change Summary |
---|---|
app/packages/state/src/hooks/useSetModalState.ts | Removed references to dynamicGroupsViewMode from the data array in the default exported function. |
app/packages/state/src/recoil/options.ts | Added import for DefaultValue , renamed dynamicGroupsViewMode to dynamicGroupsViewModeStore , updated its type to allow null , changed its default value to null , and introduced a new dynamicGroupsViewMode selectorFamily. |
dynamicGroupsViewMode
state management, which is directly related to the removal of dynamicGroupsViewMode
references in the main PR, indicating a connection in how modal state is managed.In the burrow where changes bloom,
Dynamic views now find more room.
With states that shift and atoms that play,
Our modal dance finds a new way!
Hops of joy, we celebrate,
For flexibility is truly great! 🐇✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Retain browser setting for dynamic groups mode
Summary by CodeRabbit
New Features
Bug Fixes