Skip to content

Commit

Permalink
Allow null as return value of the mount components (#78)
Browse files Browse the repository at this point in the history
  • Loading branch information
xaviervia authored Jul 14, 2022
1 parent c36a0cd commit 4f50085
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
6 changes: 3 additions & 3 deletions packages/@react-facet/core/src/components/Map.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ import { EqualityCheck, Facet, NO_VALUE } from '../types'

export type MapProps<T> = {
array: Facet<T[]>
children: (item: Facet<T>, index: number) => ReactElement
children: (item: Facet<T>, index: number) => ReactElement | null
equalityCheck?: EqualityCheck<T>
}

Expand Down Expand Up @@ -35,7 +35,7 @@ export const Map = <T,>({ array, children, equalityCheck }: MapProps<T>) => {
type MapChildMemoProps<T> = {
arrayFacet: Facet<T[]>
index: number
children: (item: Facet<T>, index: number) => ReactElement
children: (item: Facet<T>, index: number) => ReactElement | null
equalityCheck: EqualityCheck<T>
}

Expand All @@ -55,7 +55,7 @@ const MapChildMemo = <T,>({ arrayFacet, index, children, equalityCheck }: MapChi
type MapChildProps<T> = {
arrayFacet: Facet<T[]>
index: number
children: (item: Facet<T>, index: number) => ReactElement
children: (item: Facet<T>, index: number) => ReactElement | null
}

const MapChild = <T,>({ arrayFacet, index, children }: MapChildProps<T>) => {
Expand Down
2 changes: 1 addition & 1 deletion packages/@react-facet/core/src/components/Mount.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ import { Facet } from '../types'

type MountProps = {
when: Facet<boolean | undefined>
children: ReactElement
children: ReactElement | null
condition?: boolean
}

Expand Down
2 changes: 1 addition & 1 deletion packages/@react-facet/core/src/components/With.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ import { Facet, NoValue } from '../types'

type WithProps<T> = {
data: Facet<T | null | undefined>
children: (data: Facet<T>) => ReactElement
children: (data: Facet<T>) => ReactElement | null
}

const hasData = <T,>(_: Facet<T | null | undefined>, shouldRender: boolean | NoValue): _ is Facet<T> => {
Expand Down

0 comments on commit 4f50085

Please sign in to comment.