this post was submitted on 31 Dec 2024
6 points (100.0% liked)

Arctic

427 readers
1 users here now

Arctic is a Lemmy client for iOS built on pure Swift. It currently supports iOS 15+ and Lemmy v0.17+

Get the latest version on TestFlight, or check it out on the AppStore.

If you would like to support Arctic’s development, feel free to Buy Me A Coffee

founded 2 years ago
MODERATORS
CreatureSurvive@lemmy.world
6
Media improvements (self.arctic)
submitted 2 weeks ago* (last edited 2 weeks ago) by dave to c/arctic@lemmy.world
11 comments fedilink hide all child comments
 

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.
top 11 comments
sorted by: hot top controversial new old
[–] CreatureSurvive@lemmy.world 1 points 2 weeks ago (1 children)

moving them by a few pixels brings up the controls layer

I’d never noticed this before. I actually wrote a custom gesture recognizer to handle a similar issue where moving a few pixels while zoomed in would trigger the zoom out action. I should be able to swap the controls gesture out to fix this as well.

Videos when rotated often fail to scale properly to the new aspect

I admittedly have not tested media previews with device rotation. There are probably a few issues to address here. I’ll look into this.

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 is simple enough, I’ll have the video controls follow the same setting for other buttons visibility so they stay synchronized.

Dismissing a video should probably pause playback

I believe this follows the setting for autoplay of videos. If autoplay is enabled, the video will resume in the feed when dismissing the media viewer, otherwise, it will just synchronize the playback position. I could be mistaken, I’ll have to take a look at it tomorrow.

[–] dave 1 points 2 weeks ago (1 children)

All sounds good. On the media pause on dismiss, I have auto play disabled, but media continues to play on dismiss so I’m not sure it’s following the setting there…

[–] CreatureSurvive@lemmy.world 1 points 2 weeks ago (2 children)

You were correct; it was disregarding the autoplay setting when dismissing the media viewer. I’ve incorporated all of your suggestions into version 0.4.3.2, which I just released a few minutes ago. I appreciate your feedback, and please let me know if there are any problems with the update.

[–] dave 1 points 2 weeks ago (1 children)

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.

[–] CreatureSurvive@lemmy.world 1 points 2 weeks ago* (last edited 2 weeks 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.

[–] dave 1 points 2 weeks ago (2 children)

Thanks for getting to this so quickly—I’ll make separate replies as I get testing to try to keep things organised.

One the ‘zoomed image small pan’, it’s definitely better than before, but what I see now is that there’s a very small threshold below which the image doesn’t move at all, but the controls are shown. And so it can be even harder to hide them again. Let me know if you can’t reproduce that, and I’ll make a recording—but I don’t think the recorder shows the screen tap position so it might be hard to see what’s happening. The basic steps are:

  1. Zoom in to image
  2. Tap and drag a tiny amount—the image should not move
  3. Controls are shown

I also see this if the image can’t move if for example it’s very wide aspect and double-tap has left black margins top and bottom—then try dragging slightly up or down and the controls are shown.

[–] CreatureSurvive@lemmy.world 2 points 1 week ago (1 children)

Alright, I managed to resolve this. Now small pans will not trigger the controls to appear. I should be releasing an update tonight that will include these fixes.

[–] dave 1 points 1 week ago

Thank you for all of that—I’ve tried very hard, but can’t get it to misbehave at all now :) Which means the only thing be got left is the layout between feed and post, which honestly isn’t a huge deal but might be good to have an option like for link previews to keep the order consistent—see this:

feed

Vs:

post

But even now, the media experience is so much smoother and more enjoyable.

[–] CreatureSurvive@lemmy.world 1 points 2 weeks ago (1 children)

This one is a bit tricky. I’ve noticed the small threshold when dragging where nothing happens, and it can be recognized as a tap. I’m not exactly sure what is causing the dead zone for panning. I have a feeling it is due to 2 gestures waiting to get directional info to decide which one to trigger.

I’ll do some more digging and see what I can come up with. Things get rather convoluted when layering multiple gestures together.

[–] dave 1 points 1 week ago* (last edited 1 week ago) (1 children)

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

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.world 2 points 1 week 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.