8000 "Show More" button is still visible even when all items are already shown · Issue #9203 · ppy/osu · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

"Show More" button is still visible even when all items are already shown #9203

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign 8000 up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Gabixel opened this issue Jun 3, 2020 · 6 comments · Fixed by #33070
Closed

"Show More" button is still visible even when all items are already shown #9203

Gabixel opened this issue Jun 3, 2020 · 6 comments · Fixed by #33070

Comments

@Gabixel
Copy link
Contributor
Gabixel commented Jun 3, 2020

Screenshots or videos showing encountered issue:
(The example shows my profile)

Web
web image
Lazer
lazer image
@LastExceed
Copy link
Contributor

Bug originates here. The button shows up when the API returns a full page (10 items in this case) because it assumes more pages to exist (which is correct in 9 out of 10 cases). you happened to have exactly 10 elements, therefore the button showed up but didn't reveal any further items when clicked. If you check your profile again now you will notice that the bug no longer occurs because your recent section now contains 11 items.

Question is, how do we tell whether or not another page exists in order to fix this?

@bdach
Copy link
Collaborator
bdach commented Jun 5, 2020

Ideally web should be returning some kind of indication that there is more data to be read (like a cursor) if it doesn't already.

I guess you could try querying for one more item than you actually need to work around the issue, but it's a hacky solution. Other endpoints have pagination and cursors already.

@Theighlin
Copy link

This seems not to happen anymore

@bdach
Copy link
Collaborator
bdach commented May 20, 2023

@Theighlin I'm not sure how you determined that, and I'm not sure I agree. The code moved places but it still appears to have a variant of the same bug.

moreButton.FadeTo(items.Count == CurrentPage?.Limit ? 1 : 0);

@Theighlin
Copy link
Theighlin commented May 20, 2023

I verified it by checking in game, not sure what you mean by variant of the bug, but in the recent section at least, the button correcty doesn't show when there are too few events, and stops showing once they are all displayed

Right, reading above it seems it needs an exact number of recent events? Then i assume if the code wasn't changed it's still there, sorry for jumping to conclusions.

@bdach
Copy link
Collaborator
bdach commented May 20, 2023

You would have to have an exact number of items that fills the last page exactly returned to reproduce the bug. Checking a few random profiles doesn't really prove the bug is fixed.

@Gabixel Gabixel changed the title "Show more" button is visible in 'recent' user-profile section even when all events are already shown "Show More" button is still visible even when all items are already shown Jan 21, 2024
@bdach bdach marked this as a duplicate of #33069 May 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants
0