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

FAQ: Add entry discussing when messages are marked as read. #789

Merged
merged 1 commit into from
Aug 29, 2020

Conversation

neiljp
Copy link
Collaborator

@neiljp neiljp commented Aug 29, 2020

This is a slightly more detailed follow-up after #787, discussing how ZT handles marking messages read, since it varies slightly from the webapp.

This was motivated after reading threads of people asking about this in the webapp, and of course we handle it differently - so we should document this!

While this should normally be fairly straightforward to understand
(selected/sent), this also explicitly mentions deviation (visible end of
messages) and similarity (search narrows) in behavior from/with other
front-ends, and the new explore mode added in
e4af01e.
@neiljp neiljp added the area: documentation Requires changes in documentation label Aug 29, 2020
@zulipbot zulipbot added the size: S [Automatic label added by zulipbot] label Aug 29, 2020
@neiljp neiljp added the PR needs review PR requires feedback to proceed label Aug 29, 2020
Copy link
Member

@preetmishra preetmishra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@neiljp Thanks for working on this. This helps in explicating how we currently handle reading messages thoroughly.

Definitely a valuable addition. 👍

@neiljp
Copy link
Collaborator Author

neiljp commented Aug 29, 2020

@preetmishra Thanks for the feedback 👍 This is a straightforward documentation change so I'm going to merge this now.

@neiljp neiljp merged commit 1d7cb41 into zulip:master Aug 29, 2020
@neiljp neiljp added this to the Next Release milestone Aug 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: documentation Requires changes in documentation PR needs review PR requires feedback to proceed size: S [Automatic label added by zulipbot]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants