-
Notifications
You must be signed in to change notification settings - Fork 78
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
feat(ActivityCenter): Fetch notifications on scroll to bottom #12712
feat(ActivityCenter): Fetch notifications on scroll to bottom #12712
Conversation
Jenkins BuildsClick to see older builds (15)
|
@jrainville the ui is little bit laggy, i can make fetch function async for smoother UX |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, but like you said, using the async version of the call would be better
5b498fd
to
4db2a2a
Compare
4db2a2a
to
dc51178
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
dc51178
to
0adab88
Compare
0adab88
to
967a901
Compare
Close #9637
Partially fixes #12646
What does the PR do
Affected areas
Activity Center
Video of functionality
Screen.Recording.2023-11-13.at.18.38.31.mov
(on video limit of notifications is 7, default is 20)