nextcloud/spreed

UI Issue with additional actions menu

Closed this issue · 4 comments

christianlupus commented

How to use GitHub

  • Please use the 👍 reaction to show that you are affected by the same issue.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

Steps to reproduce

  1. Go to talk web app and to a chat room with some messages from you.
  2. Make sure, there is a message from you (that you can delete)
  3. Open the additional options (3 dot menu)
  4. Make sure, that the additional menu needs scrolling (change window size, resolution, or zoom factor)
  5. Try to access the menu entry to delete the chat message

Expected behaviour

Smooth scrolling and no UX issues

Actual behaviour

If you scroll down using the mouse wheel and move the mouse after that, the scroll jumps back to the top. I think this happens once the mouse touches one of the horizontal lines in the menu.

Talk app

I have no access, this was tested on the server of the Nextcloud conference 2023 running right now.

Browser

Microphone available: irrelevant

Camera available: irrelevant

Operating system: Archlinux

Browser name: Firefox

Browser version: 110.0

Browser log

Nothing was logged

Server configuration

Mostly irrelevant as UX. Also unknown as foreign server.

Nextcloud configuration:
Unknwon as foreign server

Server log (data/nextcloud.log)

Irrelevant here as pure UX

christianlupus commented

I am unsure if this issue is in the Vue components used of in the usage of them. You might need to investigate.

For Clearness, I added a screencast here:

vokoscreenNG-2023-09-17_10-36-45.mp4
Antreesy commented

Hi, Christian! This definitely looks like an issue with vue-library, and should come from this code , as removing this listener via DevTools helps:

https://github.com/nextcloud-libraries/nextcloud-vue/blob/57557706ea8141dfafee62d211d7b2810a0d19e7/src/components/NcActions/NcActions.vue#L843-L860

Can you upstream this issue to https://github.com/nextcloud/nextcloud-vue, please? And we'll also keep this open until it's solved.

christianlupus commented

I will report at the Vue library. Thanks for looking into it.

  • 👍1
nickvergessen commented

Fixed upstream and the library was updated, so assuming this is fixed.