element-hq / element-web

A glossy Matrix collaboration client for the web.
https://element.io
GNU Affero General Public License v3.0
11.25k stars 2.01k forks source link

Pinned messages links at top of timeline cycle in reverse and don't scroll to the right place #28105

Open simonmichael opened 2 months ago

simonmichael commented 2 months ago

Steps to reproduce

  1. In a room with three pinned messages
  2. I click the "N of 3 Pinned messages" area at top of timeline

Outcome

What did you expect?

I expected to cycle through the pinned messages in numeric order (123123..). And, the timeline to scroll to whichever pinned message is currently displayed.

What happened instead?

They cycle in reverse order (321321..). And the timeline seems out of step with the pinned message currently displayed at top.

Operating system

macos sonoma

Application version

element 1.11.78 or 2024092401

How did you install the app?

No response

Homeserver

No response

Will you send logs?

No

daniellekirkwood commented 1 month ago

related: https://github.com/element-hq/element-web/issues/27507

simonmichael commented 1 month ago

Update: when I looked at this on a phone today, I could see it making more sense. I guess maybe it works this way: at the top it always shows the pinned message previous to the one you're looking at in the timeline. Ie, it starts by showing the most recent pinned message, and by tapping up there you can step backward through them. I was able to understand this more easily on the small phone screen. Functioning As Designed ? Maybe a little up arrow at the top could make it more obvious ?

Aside: it would be very helpful if when you scroll to a pinned message, it was highlighted somehow. The position is unstable and it can be hard to pick out the pinned message and understand what's going on.