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

msglist: Message bounds hard to see in a run of messages from the same user #1142

Open
chrisbobbe opened this issue Dec 12, 2024 · 2 comments · May be fixed by #1152
Open

msglist: Message bounds hard to see in a run of messages from the same user #1142

chrisbobbe opened this issue Dec 12, 2024 · 2 comments · May be fixed by #1152
Labels
a-design Visual and UX design a-msglist The message-list screen, except what's label:a-content beta feedback Things beta users have specifically asked for

Comments

@chrisbobbe
Copy link
Collaborator

When we show a run of consecutive messages from the same user, it can be hard to see where one message ends and the next begins. Compare zulip-flutter and web:

web zulip-flutter
image image
image image

I noticed this at #446 (review) and it came up again in beta feedback: https://chat.zulip.org/#narrow/channel/48-mobile/topic/mobile.20beta.20feedback/near/2000252

One clear improvement, I think, is to tint the background of a message when you long-press it, which you might do to open the message action sheet. We even have a Figma frame we can follow:

image

Once message is pressed there should be a tint of the background, since the background color could be anything. So pressedTint is a transparent foreground color.

Pressed state here is kind of selected state, because we want to keep highlighting the message as a contextual information for a dialog of editing the message.

That design has a different motivation (#126 edit message) but would help this message-bounds issue too, without having to add or rearrange anything in the layout. (Space is tight on mobile.)

@chrisbobbe chrisbobbe added the beta feedback Things beta users have specifically asked for label Dec 12, 2024
@chrisbobbe chrisbobbe added the a-msglist The message-list screen, except what's label:a-content label Dec 12, 2024
@Gaurav-Kushwaha-1225

This comment was marked as off-topic.

@chrisbobbe

This comment was marked as off-topic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a-design Visual and UX design a-msglist The message-list screen, except what's label:a-content beta feedback Things beta users have specifically asked for
Projects
Status: No status
3 participants