LBRYCommunity/lbry-desktop
1
0
Fork 0
Code Issues 596 Pull requests 6 Projects Releases 238 Packages Wiki Activity

Refresh on publish page does not have loading indicator

Browse source 
## Fixes:
4214

## Approach:
`ClaimList` currently only shows the loading indicator if there is a header. We could make the indicator always appear at the top of the list when `loading=true`, but that would be the list jump up and down as the indicator appears/dissapears. I assume that is too distracting and is the reason why it was not made that way in the first place.

Show the indicator in place of the Refresh button while fetching instead.
This commit is contained in:
infiinte-persistence 2020-06-30 18:14:18 +08:00 committed by Sean Yesmunt
parent fe3ef8f8e5
commit 3288f235fb
1 changed files with 9 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
ui/page/fileListPublished/view.jsx
View file

@ -63,12 +63,15 @@ function FileListPublished(props: Props) {
title={__('Publishes')}
titleActions={
<div className="card__actions--inline">
<Button
button="alt"
label={__('Refresh')}
icon={ICONS.REFRESH}
onClick={() => fetchClaimListMine(params.page, params.page_size)}
/>
{fetching && <Spinner type="small" />}
{!fetching && (
<Button
button="alt"
label={__('Refresh')}
icon={ICONS.REFRESH}
onClick={() => fetchClaimListMine(params.page, params.page_size)}
/>
)}
<Button
icon={ICONS.PUBLISH}
button="secondary"