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

Display an empty pinned message banner when loading #28633

Closed
wants to merge 1 commit into from

Conversation

florianduros
Copy link
Member

@florianduros florianduros commented Dec 4, 2024

Checklist

  • Tests written for new code (and old code if feasible).
  • New or updated public/exported symbols have accurate TSDoc documentation.
  • Linter and other CI checks pass.
  • I have licensed the changes to Element by completing the Contributor License Agreement (CLA)

Closes #28390

@gaelledel
Copy link

tested with netlify build and it is not ideal to have the pinned msg banner empty for so long either. I wonder if we could test to only have the banner with content be displayed once it's ready and fetched? How would this feel like? (need to test)

@florianduros
Copy link
Member Author

florianduros commented Dec 4, 2024

@gaelledel I don't understand. This is the behaviour on develop/app. We display the banner when the pinned messages are fetched => jumpy banner.

@gaelledel
Copy link

gaelledel commented Dec 4, 2024

After a call with @florianduros. We agreed that it would be better to have a blank banner which would in essence be preventing timeline jumpyness in rooms with pinned messages. The timeline jumping up and down as it is currently dramatically impacts the user experience of a core section of our app. I therefore believe keeping this misbehaviour/bug will create negative perception of our product. The impact of keeping this is larger than the fix proposed by Florian. In the future, the timeline rework would ensure we don't have this bug to start with but until then, this PR is an acceptable fix. Fyi @daniellekirkwood @americanrefugee - please check that you are happy with this.

@americanrefugee
Copy link

After a call with @florianduros. We agreed that it would be better to have a blank banner which would in essence be preventing timeline jumpyness in rooms with pinned messages. The timeline jumping up and down as it is currently dramatically impacts the user experience of a core section of our app. I therefore believe keeping this misbehaviour/bug will create negative perception of our product. The impact of keeping this is larger than the fix proposed by Florian. In the future, the timeline rework would ensure we don't have this bug to start with but until then, this PR is an acceptable fix. Fyi @daniellekirkwood @americanrefugee - please check that you are happy with this.

I'm confused... The intended behavior is that the pinned messages banner displays on top of / over the timeline, not IN the the timeline. I don't understand why the timeline content would be "jumping" up or down.

@florianduros
Copy link
Member Author

Replaced by #28654

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

rooms with pinned messages scroll down and back up when you click into the room
3 participants