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

Load messages after the given day #72

Open
1 of 2 tasks
MadLittleMods opened this issue Sep 20, 2022 · 1 comment
Open
1 of 2 tasks

Load messages after the given day #72

MadLittleMods opened this issue Sep 20, 2022 · 1 comment
Labels
A-archive-room-view The view to look at a room day by day in the archive T-Defect Something isn't working. Bugs, crashes, or other reported issues.

Comments

@MadLittleMods
Copy link
Contributor

MadLittleMods commented Sep 20, 2022

Following up with the UX problems mentioned in #71, it would be really nice to be able to load messages after the given day:

UX problems

We can't show messages after the selected day

With /messages, we can only pick an event and paginate backwards because paginating forwards does not work properly with /backfill. And /context doesn't /backfill at all.

Because we can't get messages after the selected day, if the day doesn't have enough messages to fill up the screen, the URL will immediately change to a day before and there is no way to scroll low enough to select the day again except for the calendar.

-- #71

Todo

  • Create MSC for being able to /backfill forwards -> MSC4000
  • Maybe update /context/{eventId} to backfill and with the MSC, we can backfill both forwards and backwards from the given eventId. We can then use /context instead of /messages.
@MadLittleMods
Copy link
Contributor Author

Given how the archive now works by showing the end of the date/time specified in the URL looking backward up to the limit, it's less clear that we actually want to do this.

The original "want" and desire for this was to be able to render the top of the date/time specified looking forwards but that just doesn't work right now because it never backfills over federation. And because we were rendering from the top, our scroll would be nicely just at the top of timeline (no monkey business to scroll to the bottom) and the date header would be visible regardless of whether we made it sticky.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-archive-room-view The view to look at a room day by day in the archive T-Defect Something isn't working. Bugs, crashes, or other reported issues.
Projects
None yet
Development

No branches or pull requests

1 participant