Skip to content
This repository has been archived by the owner on Sep 20, 2023. It is now read-only.

Notifications paging #28

Closed
rnystrom opened this issue Jun 28, 2017 · 2 comments
Closed

Notifications paging #28

rnystrom opened this issue Jun 28, 2017 · 2 comments
Labels
🆕 feature request A feature being requested to be added, not yet approved
Milestone
1.0.0

Comments

@rnystrom
Copy link
Member

Only showing the first page of notifications content. Should add paging.
@rnystrom rnystrom added the 🆕 feature request A feature being requested to be added, not yet approved label Jun 28, 2017
@rnystrom rnystrom added this to the 1.0.0 milestone Jun 28, 2017
@rnystrom
Copy link
Member Author

Almost done

@rnystrom
Copy link
Member Author

rnystrom commented Jul 4, 2017

Done w/ a sort of weird UI. B/c I filter read and unread notifications w/ unread being the first selected (segmented control), there is a possibility where the next page of notifications only contains unread stuff, thus not updating the UI.

So instead of infinite scroll, I'm adding a cell at the bottom that shows the current page number. Then you tap it and can get UI feedback when a new page has loaded, but no new notifications show up.

I'm actually really happy about this design. Avoids any confusion.

@rnystrom rnystrom closed this as completed Jul 4, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
🆕 feature request A feature being requested to be added, not yet approved
Projects
None yet
Milestone
1.0.0
Development

No branches or pull requests
1 participant
@rnystrom