diff options
author | Hailey <me@haileyok.com> | 2024-03-08 14:31:24 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-03-08 14:31:24 -0800 |
commit | 0f9f08b1ef795215975c7b041d0e94a992d22124 (patch) | |
tree | e21d78f80631c51efc48f65574cd0999cb6cb228 /src/components/Dialog | |
parent | 8f623c3bdf8dbbdc4c4f10f19b0b2c134b4160cb (diff) | |
download | voidsky-0f9f08b1ef795215975c7b041d0e94a992d22124.tar.zst |
Fix reactivity of dialogs (Dialogs Pt. 1) (#3146)
* Improve a11y on ios * Format * Remove android * Fix android * Revert some changes * use sharedvalue for `importantForAccessibility` * add back `isOpen` * fix some more types --------- Co-authored-by: Eric Bailey <git@esb.lol>
Diffstat (limited to 'src/components/Dialog')
-rw-r--r-- | src/components/Dialog/context.ts | 6 | ||||
-rw-r--r-- | src/components/Dialog/types.ts | 2 |
2 files changed, 3 insertions, 5 deletions
diff --git a/src/components/Dialog/context.ts b/src/components/Dialog/context.ts index 9b571e8e9..859f8edd7 100644 --- a/src/components/Dialog/context.ts +++ b/src/components/Dialog/context.ts @@ -21,8 +21,7 @@ export function useDialogControl(): DialogOuterProps['control'] { open: () => {}, close: () => {}, }) - const {activeDialogs, openDialogs} = useDialogStateContext() - const isOpen = openDialogs.includes(id) + const {activeDialogs} = useDialogStateContext() React.useEffect(() => { activeDialogs.current.set(id, control) @@ -36,7 +35,6 @@ export function useDialogControl(): DialogOuterProps['control'] { () => ({ id, ref: control, - isOpen, open: () => { control.current.open() }, @@ -44,6 +42,6 @@ export function useDialogControl(): DialogOuterProps['control'] { control.current.close(cb) }, }), - [id, control, isOpen], + [id, control], ) } diff --git a/src/components/Dialog/types.ts b/src/components/Dialog/types.ts index fa9398fe0..4fc60ec39 100644 --- a/src/components/Dialog/types.ts +++ b/src/components/Dialog/types.ts @@ -22,7 +22,7 @@ export type DialogControlRefProps = { export type DialogControlProps = DialogControlRefProps & { id: string ref: React.RefObject<DialogControlRefProps> - isOpen: boolean + isOpen?: boolean } export type DialogContextProps = { |