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

Messaging: Condition Auto-scrolling #409

Open
micahalcorn opened this issue Aug 15, 2018 · 0 comments
Open

Messaging: Condition Auto-scrolling #409

micahalcorn opened this issue Aug 15, 2018 · 0 comments
Labels
dapp enhancement New feature or request javascript messaging P3 Users are not significantly affected, minor cosmetic issue ui/ux

Comments

@micahalcorn
Copy link
Member

micahalcorn commented Aug 15, 2018 •
edited
Loading

I removed the autoscroll because there are too many rerenders. It needs to be added back and conditionally fire or trigger an indicator that there are new messages below.

The conversation currently auto-scrolls whenever a new message arrives or the conversation is changed. We should do something along the lines of detecting whether or not the user has scrolled up from the bottom and then adding an element indicating that there is one or more new messages below.
@micahalcorn micahalcorn self-assigned this Aug 15, 2018
@micahalcorn micahalcorn changed the title Messaging: Conditionally Autoscroll Messaging: Condition Auto-scrolling Aug 18, 2018
@micahalcorn micahalcorn removed their assignment Mar 17, 2019
@micahalcorn micahalcorn added the P3 Users are not significantly affected, minor cosmetic issue label Apr 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
dapp enhancement New feature or request javascript messaging P3 Users are not significantly affected, minor cosmetic issue ui/ux
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@micahalcorn