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

Fuse recipient header into app bar for topic narrows #1039

Open
gnprice opened this issue Nov 1, 2024 · 1 comment
Open

Fuse recipient header into app bar for topic narrows #1039

gnprice opened this issue Nov 1, 2024 · 1 comment
Labels
a-design Visual and UX design a-msglist The message-list screen, except what's label:a-content
Milestone

Comments

@gnprice
Copy link
Member

gnprice commented Nov 1, 2024

Currently when viewing the message list for a topic narrow, we have an app bar that shows both the channel and the topic, and then a recipient header that shows the topic. (And both app bar and recipient header have the channel's subscription color.)

This looks kind of redundant. Especially so when the thread is long enough to fill the screen, so that the recipient header reaches the top — then the topic name gets displayed twice, one right below the other, at the top of the screen. It'd be good to eliminate the redundancy.

Vlad has made prototype designs for us that do that; see previous issue zulip/zulip-mobile#5511. Here's a screenshot: image

Design questions we'll want to resolve before implementing this include:

  • Have an example where the thread is shorter than the screen, so there's empty space between the app bar and the first message; make sure we're happy with how that looks, or adjust something. (Possibly switch to pinning the earliest message at the top instead of the latest at the bottom, so the empty space goes between the last message and the compose box.)
  • Decide how to handle showing the messages' dates, which currently live in the recipient header.

This is a post-launch issue; it doesn't represent any missing functionality, and I think the redundancy is OK for launch.

@gnprice gnprice added a-msglist The message-list screen, except what's label:a-content a-design Visual and UX design labels Nov 1, 2024
@gnprice gnprice added this to the Post-launch milestone Nov 1, 2024
@chrisbobbe
Copy link
Collaborator

  • Have an example where the thread is shorter than the screen, so there's empty space between the app bar and the first message; make sure we're happy with how that looks, or adjust something. (Possibly switch to pinning the earliest message at the top instead of the latest at the bottom, so the empty space goes between the last message and the compose box.)

The message list might also be empty, right, and in that case I think we don't show any recipient headers.

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
Projects
Status: No status
Development

No branches or pull requests

2 participants