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 messages links at top of timeline cycle in reverse and don't scroll to the right place #28105

Open
simonmichael opened this issue Sep 24, 2024 · 2 comments
Labels
A-Message-Pinning O-Occasional Affects or can be seen by some users regularly or most users rarely T-Enhancement

Comments

@simonmichael
Copy link

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

@dosubot dosubot bot added O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist labels Sep 24, 2024
@florianduros florianduros transferred this issue from element-hq/element-desktop Sep 26, 2024
@florianduros florianduros added T-Enhancement and removed T-Defect S-Minor Impairs non-critical functionality or suitable workarounds exist Z-Labs labels Sep 26, 2024
@daniellekirkwood
Copy link
Contributor

related: #27507

@simonmichael
Copy link
Author

simonmichael commented Sep 28, 2024

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.

@dbkr dbkr added the S-Minor Impairs non-critical functionality or suitable workarounds exist label Sep 30, 2024
@florianduros florianduros removed Z-Labs S-Minor Impairs non-critical functionality or suitable workarounds exist labels Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Message-Pinning O-Occasional Affects or can be seen by some users regularly or most users rarely T-Enhancement
Projects
None yet
Development

No branches or pull requests

4 participants