Loving the new link preview position option :)

Is there space for a couple of media tweaks?

  1. Images, once zoomed in, have this unfortunate feature where moving them by a few pixels brings up the controls layer, and then it’s hard to get rid of—single click zooms back to full image (which is great for quick dismiss). Maybe the controls could be completely disabled if the zoom is anything but ‘fit to screen’ or whatever that default state is known as?
  2. Videos when rotated often fail to scale properly to the new aspect, getting progressively more and more zoomed.
  3. Tapping a video in the feed to show it full screen leaves the playback controls visible, then another tap brings up the share / dismiss controls over the top, and then finally a third tap leaves just the video. This feels like it could be smoother—again maybe start with no controls visible?
  4. Dismissing a video should probably pause playback, as it can be hard to hit the tiny pause button back in the feed.
  • daveOP
    link
    fedilink
    English
    arrow-up
    1
    ·
    edit-2
    4 days ago

    This one still seems to be there… See this example where many times a small drag activates the controls.

    Sometimes the drag moves the image, sometimes it doesn’t (not sure if the screen recorder can capture taps / drags?).

    Would it be too much to simply disable the controls overlay when zoomed in? Just thinking that might be easier than wrangling the gestures.

    Edit: forgot to say that disabling might also solve the issue of accidental single tap before zooming, which currently leaves the controls visible and no simple way to get rid of them without zooming out again.

    • CreatureSurvive@lemmy.worldM
      link
      fedilink
      arrow-up
      2
      ·
      2 days ago

      Alright, I’ve made some progress and believe I’ve finally resolved the issue. It turned out that my previous approach to checking for movement using tap gestures was only detecting movement after the gesture had already been recognized. I’ve updated the code accordingly, and I’ve been spamming pan gestures for the past day without any issues. So far, I haven’t noticed a single instance of the controls showing up.

      As an additional layer of security, I might consider disabling the gesture while the user is zoomed, unless the tap occurs within the image margins, where the pan gesture is already not recognized.