refactor(mobile): Simplify EntryListSelector and ViewPagerList component structure
Move TimelineSelectorHeader to wrap ViewPagerList and remove redundant wrapping in EntryListSelector Signed-off-by: Innei <tukon479@gmail.com>
This commit is contained in:
parent
277e27cadc
commit
b412798958
|
|
@ -2,7 +2,6 @@ import { FeedViewType } from "@follow/constants"
|
|||
|
||||
import { EntryListContentGrid } from "@/src/modules/entry-list/EntryListContentGrid"
|
||||
|
||||
import { TimelineSelectorHeader } from "../screen/TimelineSelectorHeader"
|
||||
import { EntryListContentArticle } from "./EntryListContentArticle"
|
||||
import { EntryListContentSocial } from "./EntryListContentSocial"
|
||||
|
||||
|
|
@ -30,9 +29,5 @@ export function EntryListSelector({
|
|||
}
|
||||
}
|
||||
|
||||
return (
|
||||
<TimelineSelectorHeader>
|
||||
<ContentComponent entryIds={entryIds} />
|
||||
</TimelineSelectorHeader>
|
||||
)
|
||||
return <ContentComponent entryIds={entryIds} />
|
||||
}
|
||||
|
|
|
|||
|
|
@ -19,6 +19,7 @@ import {
|
|||
} from "@/src/store/entry/hooks"
|
||||
import { useListEntryIds } from "@/src/store/list/hooks"
|
||||
|
||||
import { TimelineSelectorHeader } from "../screen/TimelineSelectorHeader"
|
||||
import { EntryListSelector } from "./EntryListSelector"
|
||||
import { usePagerView } from "./usePagerView"
|
||||
|
||||
|
|
@ -73,29 +74,31 @@ function ViewPagerList({ viewId }: { viewId: FeedViewType }) {
|
|||
|
||||
return (
|
||||
<View style={styles.container}>
|
||||
<AnimatedPagerView
|
||||
testID="pager-view"
|
||||
ref={pagerRef}
|
||||
style={styles.PagerView}
|
||||
initialPage={page}
|
||||
layoutDirection="ltr"
|
||||
overdrag
|
||||
onPageScroll={rest.onPageScroll}
|
||||
onPageSelected={rest.onPageSelected}
|
||||
onPageScrollStateChanged={rest.onPageScrollStateChanged}
|
||||
pageMargin={10}
|
||||
orientation="horizontal"
|
||||
>
|
||||
{useMemo(
|
||||
() =>
|
||||
views.map((view) => (
|
||||
<View testID="pager-view-content" key={view.view}>
|
||||
<ViewEntryList viewId={view.view} />
|
||||
</View>
|
||||
)),
|
||||
[],
|
||||
)}
|
||||
</AnimatedPagerView>
|
||||
<TimelineSelectorHeader>
|
||||
<AnimatedPagerView
|
||||
testID="pager-view"
|
||||
ref={pagerRef}
|
||||
style={styles.PagerView}
|
||||
initialPage={page}
|
||||
layoutDirection="ltr"
|
||||
overdrag
|
||||
onPageScroll={rest.onPageScroll}
|
||||
onPageSelected={rest.onPageSelected}
|
||||
onPageScrollStateChanged={rest.onPageScrollStateChanged}
|
||||
pageMargin={10}
|
||||
orientation="horizontal"
|
||||
>
|
||||
{useMemo(
|
||||
() =>
|
||||
views.map((view) => (
|
||||
<View testID="pager-view-content" key={view.view}>
|
||||
<ViewEntryList viewId={view.view} />
|
||||
</View>
|
||||
)),
|
||||
[],
|
||||
)}
|
||||
</AnimatedPagerView>
|
||||
</TimelineSelectorHeader>
|
||||
</View>
|
||||
)
|
||||
}
|
||||
|
|
|
|||
Loading…
Reference in New Issue