Skip to content

Commit

Permalink
[C-2577] Fix mobile web social buttons by removing pull-to-refresh (#…
Browse files Browse the repository at this point in the history
  • Loading branch information
dylanjeffers committed May 5, 2023
1 parent e45a471 commit 05d108c
Show file tree
Hide file tree
Showing 6 changed files with 47 additions and 409 deletions.

This file was deleted.

This file was deleted.

This file was deleted.

Original file line number Diff line number Diff line change
@@ -1,9 +1,8 @@
import { useContext, useEffect, useCallback } from 'react'
import { useContext, useEffect } from 'react'

import {
FeedFilter,
Name,
Status,
feedPageLineupActions as feedActions
} from '@audius/common'
import cn from 'classnames'
Expand All @@ -15,8 +14,6 @@ import { HeaderContext } from 'components/header/mobile/HeaderContextProvider'
import Lineup from 'components/lineup/Lineup'
import MobilePageContainer from 'components/mobile-page-container/MobilePageContainer'
import { useMainPageHeader } from 'components/nav/store/context'
import PullToRefresh from 'components/pull-to-refresh/PullToRefresh'
import useAsyncPoll from 'hooks/useAsyncPoll'
import { FeedPageContentProps } from 'pages/feed-page/types'
import { BASE_URL, FEED_PAGE } from 'utils/route'

Expand Down Expand Up @@ -87,16 +84,6 @@ const FeedPageMobileContent = ({
record(make(Name.FEED_CHANGE_VIEW, { view: filter }))
}

const refresh = useCallback(
() => refreshFeedInView(true),
[refreshFeedInView]
)
const asyncRefresh = useAsyncPoll({
call: refresh,
variable: feed.status,
value: Status.SUCCESS
})

return (
<MobilePageContainer
title={feedTitle}
Expand All @@ -114,9 +101,7 @@ const FeedPageMobileContent = ({
[styles.playing]: !!lineupProps.playingUid
})}
>
<PullToRefresh fetchContent={asyncRefresh}>
<Lineup {...lineupProps} />
</PullToRefresh>
<Lineup {...lineupProps} />
</div>
</MobilePageContainer>
)
Expand Down
Loading

0 comments on commit 05d108c

Please sign in to comment.