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

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?

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?
Videos when rotated often fail to scale properly to the new aspect, getting progressively more and more zoomed.
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?
Dismissing a video should probably pause playback, as it can be hard to hit the tiny pause button back in the feed.

you are viewing a single comment's thread
view the rest of the comments

[–] CreatureSurvive@lemmy.world 1 points 2 weeks ago (10 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 (9 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 (8 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.

load more comments (6 replies)