Simple Slideshow

[printfuck]

This is a simple slideshow feature with a setting for the time interval.

I also admittedly changed the icon placement inside the buttons - this is not necessary, but it made the alignment nicer (in my opinion).

firefox_4th9rIfvZ9.mp4

[codecov]

Codecov Report

Attention: Patch coverage is 80.45113% with 26 lines in your changes are missing coverage. Please review.

Project coverage is 57.53%. Comparing base (b2d591b) to head (66eebf8).
Report is 7 commits behind head on master.

❗ Current head 66eebf8 differs from pull request most recent head c877290

Please upload reports for the commit c877290 to get more accurate results.

Files	Patch %	Lines
...omponents/photoGallery/presentView/icons/Pause.tsx	13.04%	20 Missing ⚠️
...toGallery/presentView/PresentNavigationOverlay.tsx	93.10%	6 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #931      +/-   ##
==========================================
+ Coverage   57.34%   57.53%   +0.18%     
==========================================
  Files         196      198       +2     
  Lines       15607    15738     +131     
  Branches      533      542       +9     
==========================================
+ Hits         8950     9055     +105     
- Misses       6408     6434      +26     
  Partials      249      249              

Flag	Coverage Δ
api	25.18% <ø> (ø)
ui	70.11% <80.45%> (+0.11%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[jordy2254]

Thanks for your PR. Sounds like a useful feature. I will checkout your changes during the week to review them.

[kkovaletp]

I like the slideshow feature from the functionality point of view. It will be definitely useful for many users.
Sure, we could start with something simple and then enhance it iteratively.
@printfuck, it would be great if you could describe it with some more details and probably attach some screenshots, so we could clearly understand all the functions and limitations: for instance, is it possible to start it for an album or it can run for all the gallery? will it play videos or skip them? etc...

[printfuck]

I like the slideshow feature from the functionality point of view. It will be definitely useful for many users. Sure, we could start with something simple and then enhance it iteratively. @printfuck, it would be great if you could describe it with some more details and probably attach some screenshots, so we could clearly understand all the functions and limitations: for instance, is it possible to start it for an album or it can run for all the gallery? will it play videos or skip them? etc...

Thanks for your reply.
It basically just iterates on all images inside of an album from within the presentView; videos won't be started. My focus was entirely on images, but I think it would make sense to play each video once and cycle by default. I think, there would be need for another switch to change video playback behaviour.

Possible options:

• only Images
• only Videos, played once
• both, videos played once

[kkovaletp]

Thanks for the explanation and the video attachment with the demo!

there would be need for another switch to change video playback behaviour.

Possible options:

• only Images
• only Videos, played once
• both, videos played once

I like the idea of having 1 more setting for this.
However, there is a tricky moment: according to your demo video, the timeout setting is controlled by an on-screen button, shown for the currently opened media. Then the new switch setting needs to be consistent with the timeout setting, it could be a 3-stages-button near the timeout one with:

• picture icon (default)
• video icon
• an icon with both picture and video icons, located as a stack or sequence
  corresponding to proposed states. But here we might confuse users, as showing these buttons on screen while a media is shown automatically sets an expectation for a user that the setting is connected to the context of the current media:
• in the case of the currently shown picture user sets the video only, what is expected and actually performed? what if the current album doesn't contain videos - should we check for that in advance and not allow users to change the setting? how might this check impact the performance of starting the view?
• in the opposite case, should we set the 'videos only' value in advance and disable the setting?

While the described scenario looks consistent to me, I'd like to read your and @jordy2254 opinions, and probably start some discussion to make sure that we understand all possible use cases and design the workflow in an optimal way)