Skip to content
This repository has been archived by the owner on Dec 15, 2022. It is now read-only.

Streaming notifications are added to bottom of notifications view #18

Closed
martensitingale opened this issue May 24, 2018 · 1 comment
Closed

Comments

@martensitingale
Copy link
Contributor

They should appear at the top, of course.

I think this is due to NotificationView.prepend actually appending the status to the end of the list. It seems that TimelineView.prepend, similarly, actually appends the status to the end of the list, but unlike the notification view it has a boolean argument first which when passed true will prepend instead of appending.

Maybe appending/prepending to a view should be factored into a simple implementation shared between all views?

@bleakgrey
Copy link
Owner

Disregard function names, they're a relic from an early alpha. I'm going to restructure all of this mess in my multiple-accounts branch anyway.

Looks like I forgot to copy the Home timeline behavior for new data. Whoops.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants