Commit d24c62ec authored by aballerr's avatar aballerr Committed by GitHub

fix: Minor polish fixes (#5407)

* Loading state fixes, make volume lowercase, remove loading circle
parent 2ea70d18
......@@ -185,7 +185,7 @@ export const CollectionNftsLoading = () => (
export const CollectionNftsAndMenuLoading = () => (
<InfiniteScrollWrapper>
<Column alignItems="flex-start" position="relative" width="full">
<Row marginY="12" gap="12">
<Row marginY="12" gap="12" marginBottom="40">
<Box className={loadingAsset} borderRadius="12" width={{ sm: '44', md: '100' }} height="44" />
<Box
className={loadingAsset}
......
import Column from 'components/Column'
import Row from 'components/Row'
import { Box } from 'nft/components/Box'
import { useIsMobile } from 'nft/hooks'
import { BannerWrapper, CollectionBannerLoading } from 'nft/pages/collection'
import { ScreenBreakpointsPaddings } from 'nft/pages/collection/index.css'
......@@ -14,10 +13,6 @@ const CollectionDescriptionSection = styled(Column)`
${ScreenBreakpointsPaddings}
`
const CollectionAssets = styled(Box)`
${ScreenBreakpointsPaddings}
`
const StyledColumn = styled(Column)`
width: 100%;
`
......@@ -38,9 +33,7 @@ export const CollectionPageSkeleton = () => {
<CollectionStatsLoading isMobile={isMobile} />
<StyledRow>{ActivitySwitcherLoading}</StyledRow>
</CollectionDescriptionSection>
<CollectionAssets>
<CollectionNftsAndMenuLoading />
</CollectionAssets>
<CollectionNftsAndMenuLoading />
</StyledColumn>
)
}
......@@ -306,7 +306,7 @@ const StatsRow = ({ stats, isMobile, ...props }: { stats: GenieCollection; isMob
</StatsItem>
) : null}
{stats.stats?.total_volume ? (
<StatsItem label="Total Volume" shouldHide={false}>
<StatsItem label="Total volume" shouldHide={false}>
{totalVolumeStr} ETH
</StatsItem>
) : null}
......
......@@ -252,7 +252,15 @@ export default function App() {
{nftFlag === NftVariant.Enabled && (
<>
<Route path="/nfts" element={<NftExplore />} />
<Route
path="/nfts"
element={
// TODO: replace loading state during Apollo migration
<Suspense fallback={null}>
<NftExplore />
</Suspense>
}
/>
<Route
path="/nfts/asset/:contractAddress/:tokenId"
element={
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment