element-hq / element-meta

Shared/meta documentation and project artefacts for Element clients
66 stars 11 forks source link

[Prep] Experiment room list filter #2270

Closed manuroe closed 5 months ago

manuroe commented 5 months ago

Description

We have several concepts and we need to be able to see the behavior on real data within the app. We are going to experiment it in specific builds of the apps Here are the designs

Acceptance criteria

Size estimate

M

Dependencies

Out of scope

Open questions

Subtasks

### Android
 - [ ] https://github.com/element-hq/element-x-android/issues/2232
 - [ ] https://github.com/element-hq/element-x-android/issues/2233
 - [ ] https://github.com/element-hq/element-x-android/issues/2234
### iOS
 - [ ] https://github.com/element-hq/element-x-ios/issues/2339
 - [ ] https://github.com/element-hq/element-x-ios/issues/2340
 - [ ] https://github.com/element-hq/element-x-ios/issues/2341
### Rust
 - [ ] https://github.com/matrix-org/matrix-rust-sdk/issues/3020
 - [ ] https://github.com/matrix-org/matrix-rust-sdk/issues/3021
 - [ ] https://github.com/matrix-org/matrix-rust-sdk/issues/3022
### Other
 - [ ] Make people test it and manage feedbacks
Velin92 commented 5 months ago

@Hywan today actually asked a very good question. Wha do we consider as "Unread" do we want to consider Unread Notifications or Unread Messages? If we consider Unread notifications, we will only display rooms that have unread content that respects the global or the custom push rules (so a mentions only will be displayed only if it has unread mentions,. but not if it has unread messages) If we consider Unread messages, we will display any room that simply has unread messages regardless if it is muted, is in mentions only and so on.

So how this filter should behave? @VolkerJunginger

VolkerJunginger commented 5 months ago

Unread rooms are those that have been triggered by a "green badge" notification. Also setting a room manually as "unread" triggers a "green" badge badge and should be shown here.

Hywan commented 5 months ago

@Velin92 I'm not sure the issue is closed ^^. Is it?

Velin92 commented 5 months ago

@Velin92 I'm not sure the issue is closed ^^. Is it?

Me and @manuroe decided to close it since we migrated to the final well defined story here: https://github.com/element-hq/element-meta/issues/2283

Everything has already been moved

Hywan commented 5 months ago

Great, thanks!