Skip to content
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 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

Pagination of online replays #1764

Closed
Aclrian opened this issue Jun 9, 2020 · 2 comments
Closed

Pagination of online replays #1764

Aclrian opened this issue Jun 9, 2020 · 2 comments
Assignees

Comments

@Aclrian
Copy link
Contributor

Aclrian commented Jun 9, 2020

Scrolling througth a huge size of replays is frustrating and replays older than 1 year are less relevant:
add a checkbox for only Replays from the last year
and a Pagination for a better overview

@norraxx
Copy link
Collaborator

norraxx commented Jun 10, 2020

Or we could make "pages" not scrolling. Less memory consumption, faster client behavior.

@1-alex98
Copy link
Member

@norraxx pages are the suggestion...

Aclrian added a commit that referenced this issue Jun 29, 2020
missing: max page count and pre-defined search doesn't work with current pagination
Aclrian added a commit that referenced this issue Jul 2, 2020
missing: max page count and pre-defined search doesn't work with current pagination
jofaul pushed a commit that referenced this issue Jul 6, 2020
missing: max page count and pre-defined search doesn't work with current pagination
jofaul added a commit that referenced this issue Jul 6, 2020
…ination' into feature/#1764-online-replays-pagination

# Conflicts:
#	src/main/java/com/faforever/client/replay/OnlineReplayVaultController.java
#	src/main/resources/theme/vault/replay/online_replays.fxml
jofaul added a commit that referenced this issue Jul 6, 2020
…ination' into feature/#1764-online-replays-pagination

# Conflicts:
#	src/main/java/com/faforever/client/api/FafApiAccessor.java
#	src/main/java/com/faforever/client/api/FafApiAccessorImpl.java
#	src/main/java/com/faforever/client/api/MockFafApiAccessor.java
#	src/main/java/com/faforever/client/remote/FafService.java
jofaul added a commit that referenced this issue Jul 6, 2020
…ination' into feature/#1764-online-replays-pagination

# Conflicts:
#	src/main/java/com/faforever/client/replay/OnlineReplayVaultController.java
jofaul pushed a commit that referenced this issue Jul 10, 2020
missing: max page count and pre-defined search doesn't work with current pagination
jofaul added a commit that referenced this issue Jul 10, 2020
…ination' into feature/#1764-online-replays-pagination

# Conflicts:
#	src/main/java/com/faforever/client/replay/OnlineReplayVaultController.java
jofaul added a commit that referenced this issue Jul 10, 2020
…ination' into feature/#1764-online-replays-pagination
jofaul added a commit that referenced this issue Jul 10, 2020
…eature/#1764-online-replays-pagination

# Conflicts:
#	src/main/java/com/faforever/client/replay/OnlineReplayVaultController.java
jofaul added a commit that referenced this issue Jul 14, 2020
jofaul added a commit that referenced this issue Jul 15, 2020
Fixes #1764

Co-Authored-By: tarmandan <58087411+tarmandan@users.noreply.github.com>
Co-Authored-By: aclrian <32142988+aclrian@users.noreply.github.com>
Co-Authored-By: Jia Hong Frenki Zhang <64331094+frezha@users.noreply.github.com>
Aclrian added a commit that referenced this issue Jul 22, 2020
Fixes #1764

Co-Authored-By: tarmandan <58087411+tarmandan@users.noreply.github.com>
Co-Authored-By: aclrian <32142988+aclrian@users.noreply.github.com>
Co-Authored-By: Jia Hong Frenki Zhang <64331094+frezha@users.noreply.github.com>
1-alex98 pushed a commit that referenced this issue Jul 25, 2020
Fixes #1764

Co-Authored-By: tarmandan <58087411+tarmandan@users.noreply.github.com>
Co-Authored-By: aclrian <32142988+aclrian@users.noreply.github.com>
Co-Authored-By: Jia Hong Frenki Zhang <64331094+frezha@users.noreply.github.com>
mrchris2000 pushed a commit to mrchris2000/downlords-faf-client that referenced this issue Apr 15, 2022
Fixes FAForever#1764

Co-Authored-By: tarmandan <58087411+tarmandan@users.noreply.github.com>
Co-Authored-By: aclrian <32142988+aclrian@users.noreply.github.com>
Co-Authored-By: Jia Hong Frenki Zhang <64331094+frezha@users.noreply.github.com>
Co-authored-by: IAmAlife <54673768+IAmAlife@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants