element-ios icon indicating copy to clipboard operation
element-ios copied to clipboard

Threads: When read, doesn't leave Unreads filter view

Open daniellekirkwood opened this issue 3 years ago • 10 comments

I got a thread message in a room, once I read it and went to the Unreads tab the message is read but the room shows in the Unreads tab still

image

daniellekirkwood avatar Nov 18 '22 14:11 daniellekirkwood

At random cases the fetcher predicate fetches data with s_hasAnyUnread:false even when the predicate has MXRoomSummaryDataTypesUnread

toshanmugaraj avatar Dec 08 '22 05:12 toshanmugaraj

At random cases the fetcher predicate fetches data with s_hasAnyUnread:false even when the predicate has MXRoomSummaryDataTypesUnread

I think it was because s_hasAnyUnread wasn't really used for this kind of fetch (very counterintuitively). Now it should after this. Please have look if you wish!

alfogrillo avatar Feb 22 '23 11:02 alfogrillo

Reopening as the original fix was reverted in https://github.com/matrix-org/matrix-ios-sdk/pull/1736.

Johennes avatar Mar 13 '23 18:03 Johennes

I don't understand why https://github.com/matrix-org/matrix-ios-sdk/pull/1723 was reverted by https://github.com/matrix-org/matrix-ios-sdk/pull/1736 ?

Unread tabs list a lot of totally read rooms.

If i locally reapply https://github.com/matrix-org/matrix-ios-sdk/pull/1723 then unread tab only lists rooms with unread messages as attended.

Please, reapply https://github.com/matrix-org/matrix-ios-sdk/pull/1723

NicolasBuquet avatar Oct 26 '23 17:10 NicolasBuquet

If I remember correctly, whilst the mentioned PR did fix the underlying bug, it completely changed the desired behaviour of the tab, which is to only show rooms with notifications, and not simply any room where the read marker isn't on the last event. This is why the bug still stands, it will need to be fixed some other way.

pixlwave avatar Nov 02 '23 17:11 pixlwave

@NicolasBuquet I have the same memories as Doug. Unintuitively although the tab says "Unreads", people expect to see rooms with notifications there. This is why we decided to revert that PR.

alfogrillo avatar Nov 03 '23 15:11 alfogrillo

Thank you for your comments.

But to have a better understanding, what is the difference between a room with unread messages and a room with notifications?

And, won't it be a good thing finally to be able to parametrize this filter on the Matrix SDK side?

NicolasBuquet avatar Nov 03 '23 16:11 NicolasBuquet

If you set a room's notification settings to Mentions and Keywords only, then it will have an unread message for you when someone sends a message of Hello world, but when someone mentions @NicolasBuquet in a message, then it will have a notification. In practice the app:

  • Shows no indicator for a read room
  • Shows a black indicator on the leading edge of the cell for an unread room
  • Shows a number on the trailing edge of the cell for a room with a notification.

Screenshot 2023-11-06 at 10 58 56 am

pixlwave avatar Nov 06 '23 11:11 pixlwave

@pixlwave Thank you for this explanation.

It's clearer for me now. 🙏

NicolasBuquet avatar Nov 06 '23 11:11 NicolasBuquet

I perpetually have a half dozen rooms listed in the "unread" filter, despite no unread notifications and nothing visible in the room unread. Tap-hold does not show "mark as read" as an option. Selecting "mark all as read" or "clear cache" in settings also has no effect.

Is there any workaround to actually get these rooms out of the "unread" filter list? Is this issue still considered a live bug or should I file a new one?

ajkessel avatar Dec 30 '24 15:12 ajkessel