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.
  • @[email protected]OPEnglish
    1•8 days ago

    On the video side, pause on dismiss is working perfectly, but I did see 2 other problems (great!). One is the zoom on rotate issue—I still see that on first rotation, although it doesn’t get ‘worse’. It ends up showing by about half the media.

    But in testing that I noticed that scrubbing the paused video in either full screen or feed view does not appear to affect the playback position where unpaused again—it restarts from where it previously was, not the new position.

    Again let me know if recordings would help illustrate.

    • @CreatureSurviveM
      1•8 days ago

      Well, one down at least.

      I took another look at the video rotation this morning. I had made a last minute change before submitting the update yesterday that I thought was harmless, but turns out it broke the rotation fix. I reworked this a bit and it seems to be working fine now.

      I completely overlooked saving the playback position while scrubbing if the video is paused. I’ll add this in the next release.

      [edit] I added support for saving media resume positions while scrubbing. I’ll still need to do a bit more testing to make sure it is working correctly.