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

Pinned message drawer is visually indifferentiable from normal messages #17213

Closed
Tracked by #1624
HarHarLinks opened this issue May 9, 2021 · 0 comments · Fixed by matrix-org/matrix-react-sdk#6096
Closed
Tracked by #1624

Comments

@HarHarLinks
Copy link
Contributor

Is your suggestion related to a problem? Please describe.

Message pinning is currently a labs feature, however it has become somewhat usable more or less recently.
The pinned message drawer is visually almost indistinguishable from the normal chat. It is the same size and at least in dark mode uses the same background color. There are no borders. The scrollbar's lower limit isn't indicated. Only the message layout itself is slightly different (but also needs further work to be pretty). The drawer title text "Pinned Messages" should stay on screen while scrolling.

Describe the solution you'd like.

  • draw clear borders to indicate what is the drawer
    • this automatically also hints at how far you can scroll
  • don't scroll the panel title
  • improve the pinned messages layout incl sender name, always display date, don't stick right up to the panel border....
  • similar UX to search, including where you can click and what happens when you do click
  • auto-close after clicking jump to(?)

Describe alternatives you've considered.

Looking at discord, they do a lot of these things, but i don't like the very narrow popup bubble approach.

Additional context

image

Version information

  • Platform: desktop

For the desktop app:

  • OS: Arch
  • Version: aur/element-desktop-nightly-bin 2021050701-1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants