transfem-org / Sharkey

🌎 A Sharkish microblogging platform 🚀
https://joinsharkey.org/
74 stars 19 forks source link

bug: Unable to scroll through long image caption indirectly #160

Open irfanhakim-as opened 9 months ago

irfanhakim-as commented 9 months ago

💡 Summary

As per title: "Unable to scroll through long image caption indirectly". I wish I could describe it better but what I mean by indirectly is by hovering or pressing on the location where the image caption is, and scrolling via scroll wheel, or trackpad - neither of them works. Scrolling only works when scrolling "directly" or "actively" via the scroll bar while on desktop/mobile, or via touch (at least on mobile).

🥰 Expected Behavior

🤬 Actual Behavior

📝 Steps to Reproduce

  1. Load this note on a Sharkey server (i.e. kitsunes.club): https://kitsunes.club/notes/9mackodtqe.
  2. Click/tap on the image to open the detailed image preview.
  3. Hover/press on the image caption area at the bottom and scroll using touch, scroll wheel, or trackpad.
  4. If that method indeed does not work, try pressing and scrolling using the provided scroll bar instead to verify that this different method does work.

💻 Frontend Environment

Device 1 (Desktop)

* Model and OS of the device(s): AMD Ryzen + NVIDIA GPU PC, EndeavourOS Linux x86_64, 6.6.1-arch1-1
* Browser: Mozilla Firefox 119.0.1 (64-bit), Google Chrome 119.0.6045.159 (Official Build) unknown (64-bit)
* Server URL: kitsunes.club
* Misskey: Sharkey 2023.11.0

Device 2 (Mobile)

* Model and OS of the device(s): iPhone 11, iOS 17.1
* Browser: Mozilla Firefox 120.0 (as PWA)
* Server URL: kitsunes.club
* Misskey: Sharkey 2023.11.0

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service:
* Misskey:
* Node:
* PostgreSQL:
* Redis:
* OS and Architecture:
Mar0xy commented 9 months ago

This could actually be two factors it could be a issue with photoswipe (the module that is used for displaying those views) https://photoswipe.com/

Or by the custom CSS Misskey applies to it https://github.com/transfem-org/Sharkey/blob/19fc5a74f0ac519781c6b0b0bbc724baa7366d8d/packages/frontend/src/components/MkMediaList.vue#L366-L397

As no one on our team really knows CSS it would be of help if some other people could look at this.

irfanhakim-as commented 9 months ago

Noted. I've also updated the issue to update the fact that it does work via touch, at least on mobile. Not sure if it works via touch on desktop.

On my end so far with the Frontend Environment I'm testing, scrolling only does not work using the mouse scroll wheel or trackpad, while on desktop.