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

msglist: Control scroll position on renarrowing for bulk message moves #1009

Open
chrisbobbe opened this issue Oct 19, 2024 · 0 comments
Open
Labels
a-msglist The message-list screen, except what's label:a-content
Milestone

Comments

@chrisbobbe
Copy link
Collaborator

chrisbobbe commented Oct 19, 2024

When you're in a narrow and messages are moved in bulk, we renarrow the message list page to the messages' destination. (#1008 would remove an uncommon case where we do this undesirably.)

When we renarrow, we should try to keep the messages the user was reading in view, preserving their scroll position when we can. Currently, we just open the new narrow at its latest messages. (This might change with #80, but that's still not the behavior described here.)

To fix, we can choose some onscreen message that was moved and try to maintain its scroll position across the move. For how we might choose that message, see #1008.

Marking as post-launch because we don't do this in zulip-mobile.

This is different from #83; that issue doesn't address renarrowing.

@chrisbobbe chrisbobbe added the a-msglist The message-list screen, except what's label:a-content label Oct 19, 2024
@chrisbobbe chrisbobbe added this to the Post-launch milestone Oct 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a-msglist The message-list screen, except what's label:a-content
Projects
Status: No status
Development

No branches or pull requests

1 participant