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

Mark all as Read (including Threads) #577

Closed
daniellekirkwood opened this issue Sep 2, 2022 · 5 comments
Closed

Mark all as Read (including Threads) #577

daniellekirkwood opened this issue Sep 2, 2022 · 5 comments
Assignees
Labels
A-Threads T-Enhancement Z-Confidence-High High confidence in the enhancement or suggestion based on known factors, or as described. Z-Labs

Comments

@daniellekirkwood
Copy link
Contributor

While this issue is marked as an enhancement we need to define the behaviour for Threads Exit Beta and then update this WIP issue with the appropriate next steps that are not blocking to the Exit Beta release.

Now that we have Thread Timelines as well as the Main Timeline we do not have the ability to mark a room as read on Web. The Main Timeline button that (without threads) marks a room as read, only applies to the main timeline.

Screen Shot 2022-09-02 at 12 43 45

On Mobile we have a room level control that marks a room as read (including threads?)

@daniellekirkwood daniellekirkwood added Team: Delight A-Threads T-Enhancement Z-Confidence-High High confidence in the enhancement or suggestion based on known factors, or as described. labels Sep 2, 2022
@daniellekirkwood daniellekirkwood self-assigned this Sep 2, 2022
@github-actions github-actions bot added the Z-Labs label Sep 2, 2022
@germain-gg
Copy link

We have a solution to mark the room as read technically, but it won't scale very well.
Follow the MSC conversation about how to do it in a more performant manner https://github.com/matrix-org/matrix-spec-proposals/pull/3773/files#r961753262

@daniellekirkwood
Copy link
Contributor Author

Related:
element-hq/element-web#24053

@clokep
Copy link
Contributor

clokep commented Jan 31, 2023

I think this work is done -- Element Web and mobile clients can mark an entire room as read including all threads in it.

@marcusmueller
Copy link

marcusmueller commented Apr 22, 2023

@clokep I'm sorry, I'll have to deny that – element web 1.11.27 doesn't mark threads read:

markread.mp4

@germain-gg
Copy link

I +1 @clokep . This has been done, and therefore will close this ticket.

@marcusmueller it sounds like you've faced a bug, and it would be good if you could open an issue and share a rageshake so we can investigate this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Threads T-Enhancement Z-Confidence-High High confidence in the enhancement or suggestion based on known factors, or as described. Z-Labs
Projects
None yet
Development

No branches or pull requests

4 participants