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

Review types of the last message displayed in room lists #4451

Closed
manuroe opened this issue Jun 21, 2021 · 6 comments · Fixed by matrix-org/matrix-ios-kit#915
Closed

Review types of the last message displayed in room lists #4451

manuroe opened this issue Jun 21, 2021 · 6 comments · Fixed by matrix-org/matrix-ios-kit#915
Assignees
Labels
A-Room-List parity-with-android parity-with-web X-Needs-Design May require input from the design team Z-Impact-2 Z-Papercuts Visible. Impactful. Predictable to action. Z-Visibility-1

Comments

@manuroe
Copy link
Member

manuroe commented Jun 21, 2021

Element Android and Web do not show join events for example.

The web filter is available at https://github.com/matrix-org/matrix-react-sdk/blob/7ab31932ce99302446b24741ac6e4ee88587766f/src/stores/room-list/MessagePreviewStore.ts#L35

iOS displays much more of them but not reactions.
We need to synchronise with the design team to get the right list of types to display.

@manuroe manuroe changed the title Review types of the last message displayed in room list Review types of the last message displayed in room lists Jun 21, 2021
@ShadowJonathan
Copy link
Contributor

I'd also like to mention that Android doesn't bump the room list on joins, this creates a counterintuitive situation where a room can be 'read' yet still appear "recent", while unread rooms are below it. I think a common usage pattern is to assess the list of recent unread rooms in one go, and stop at the first unread.

@pixlwave
Copy link
Contributor

pixlwave commented Jul 21, 2021

For clarity web only shows:

Messages - m.room.message
Call Invite - m.call.invite
Call Answer - m.call.answer
Call Hangup - m.call.hangup
Sticker - m.sticker
Reactions (👍) - m.reaction

@niquewoodhouse niquewoodhouse added the X-Needs-Design May require input from the design team label Aug 17, 2021
@manuroe manuroe added this to To do in iOS Papercuts Sep 10, 2021
@manuroe manuroe moved this from To do to To do 2 in iOS Papercuts Sep 10, 2021
@manuroe manuroe moved this from To do to Backlog in iOS Papercuts Sep 10, 2021
@pixlwave pixlwave moved this from To do to Need design in iOS Papercuts Sep 10, 2021
@niquewoodhouse
Copy link

I think for consistency, we should follow web and android. I did want to discuss this with other designers but I realise the scope of this work is papercuts for iOS. If we change it for iOS here, we won't be making the product less papercutty, we'll just be making it more inconsistent, and for an unknown length of time, because it's not clear when web/android would adapt to a new system.

@pixlwave pixlwave moved this from Need design to Ready to develop in iOS Papercuts Sep 14, 2021
@pixlwave pixlwave moved this from Ready to develop to In progress in iOS Papercuts Sep 23, 2021
@pixlwave pixlwave self-assigned this Sep 23, 2021
@pixlwave
Copy link
Contributor

Note - Reactions are not actually shown on web due to a setting that defaults to false here:
https://github.com/matrix-org/matrix-react-sdk/blob/bbc8e4da23d1a3313839ca49bcf0549ccd1d9000/src/settings/Settings.tsx#L230

As these aren't currently shown on iOS, I won't introduce them into the mix.

@pixlwave
Copy link
Contributor

pixlwave commented Sep 29, 2021

One final tweak - without including create room events, a room with no content appears out of order and has blank labels, so I'm going to add that one back in.

Example without it:
Simulator Screen Shot - iPhone 12 mini - 2021-09-29 at 13 22 15

After discussion we decided to replace the create room event's string with "You joined" for the time being and will review that decision in #4918.

iOS Papercuts automation moved this from In review to Done Sep 30, 2021
@pixlwave
Copy link
Contributor

Additionally added back m.room.encrypted to show unable to decrypt errors when keys are missing, otherwise the same situation as above could occur.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Room-List parity-with-android parity-with-web X-Needs-Design May require input from the design team Z-Impact-2 Z-Papercuts Visible. Impactful. Predictable to action. Z-Visibility-1
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

4 participants