filter out videos, or shuffle the media at slideshow when appropriate

app/src/main/kotlin/com/simplemobiletools/gallery/activities/ViewPagerActivity.kt

@@ -242,8 +242,8 @@ class ViewPagerActivity : SimpleActivity(), ViewPager.OnPageChangeListener, View
         return true
     }
 
-    private fun updatePagerItems() {
+    private fun updatePagerItems(media: MutableList<Medium>) {
-        val pagerAdapter = MyPagerAdapter(this, supportFragmentManager, mMedia.toMutableList())
+        val pagerAdapter = MyPagerAdapter(this, supportFragmentManager, media)
         if (Build.VERSION.SDK_INT < Build.VERSION_CODES.JELLY_BEAN_MR1 || !isDestroyed) {
             view_pager.apply {
                 adapter = pagerAdapter
@@ -261,12 +261,14 @@ class ViewPagerActivity : SimpleActivity(), ViewPager.OnPageChangeListener, View
     }
 
     private fun startSlideshow() {
-        hideSystemUI()
+        if (getMediaForSlideshow()) {
-        mSlideshowInterval = config.slideshowInterval
+            hideSystemUI()
-        mSlideshowMoveBackwards = config.slideshowMoveBackwards
+            mSlideshowInterval = config.slideshowInterval
-        mIsSlideshowActive = true
+            mSlideshowMoveBackwards = config.slideshowMoveBackwards
-        scheduleSwipe()
+            mIsSlideshowActive = true
-        window.addFlags(WindowManager.LayoutParams.FLAG_KEEP_SCREEN_ON)
+            scheduleSwipe()
+            window.addFlags(WindowManager.LayoutParams.FLAG_KEEP_SCREEN_ON)
+        }
     }
 
     private fun stopSlideshow() {
@@ -294,6 +296,29 @@ class ViewPagerActivity : SimpleActivity(), ViewPager.OnPageChangeListener, View
         }
     }
 
+    private fun getMediaForSlideshow(): Boolean {
+        var slideshowMedia = mMedia.toMutableList()
+        if (!config.slideshowIncludeVideos) {
+            slideshowMedia = slideshowMedia.filter { it.isImage() || it.isGif() } as MutableList
+        }
+
+        if (config.slideshowRandomOrder) {
+            Collections.shuffle(slideshowMedia)
+            mPos = 0
+        } else {
+            mPath = getCurrentPath()
+            mPos = getPositionInList(slideshowMedia)
+        }
+
+        return if (slideshowMedia.isEmpty()) {
+            toast(R.string.no_media_for_slideshow)
+            false
+        } else {
+            updatePagerItems(slideshowMedia)
+            true
+        }
+    }
+
     private fun copyMoveTo(isCopyOperation: Boolean) {
         val files = ArrayList<File>(1).apply { add(getCurrentFile()) }
         tryCopyMoveFilesTo(files, isCopyOperation) {
@@ -527,21 +552,21 @@ class ViewPagerActivity : SimpleActivity(), ViewPager.OnPageChangeListener, View
         mPrevHashcode = media.hashCode()
         mMedia = media
         if (mPos == -1) {
-            mPos = getProperPosition()
+            mPos = getPositionInList(media)
         } else {
             mPos = Math.min(mPos, mMedia.size - 1)
         }
 
         updateActionbarTitle()
-        updatePagerItems()
+        updatePagerItems(mMedia.toMutableList())
         invalidateOptionsMenu()
         checkOrientation()
     }
 
-    private fun getProperPosition(): Int {
+    private fun getPositionInList(items: MutableList<Medium>): Int {
         mPos = 0
         var i = 0
-        for (medium in mMedia) {
+        for (medium in items) {
             if (medium.path == mPath) {
                 return i
             }

app/src/main/res/values-cs/strings.xml

@@ -87,6 +87,7 @@
     <string name="use_fade">Use fade animations</string>
     <string name="move_backwards">Move backwards</string>
     <string name="slideshow_ended">The slideshow ended</string>
+    <string name="no_media_for_slideshow">No media for the slideshow have been found</string>
 
     <!-- Settings -->
     <string name="show_hidden_media">Zobrazit skryté média</string>

app/src/main/res/values-de/strings.xml

@@ -87,6 +87,7 @@
     <string name="use_fade">Use fade animations</string>
     <string name="move_backwards">Move backwards</string>
     <string name="slideshow_ended">The slideshow ended</string>
+    <string name="no_media_for_slideshow">No media for the slideshow have been found</string>
 
     <!-- Settings -->
     <string name="show_hidden_media">Versteckte Ordner zeigen</string>

app/src/main/res/values-es/strings.xml

@@ -87,6 +87,7 @@
     <string name="use_fade">Use fade animations</string>
     <string name="move_backwards">Move backwards</string>
     <string name="slideshow_ended">The slideshow ended</string>
+    <string name="no_media_for_slideshow">No media for the slideshow have been found</string>
 
     <!-- Settings -->
     <string name="show_hidden_media">Mostrar carpetas ocultas</string>

app/src/main/res/values-fr/strings.xml

@@ -87,6 +87,7 @@
     <string name="use_fade">Use fade animations</string>
     <string name="move_backwards">Move backwards</string>
     <string name="slideshow_ended">The slideshow ended</string>
+    <string name="no_media_for_slideshow">No media for the slideshow have been found</string>
 
     <!-- Settings -->
     <string name="show_hidden_media">Afficher les dossiers cachés</string>

app/src/main/res/values-hu/strings.xml

@@ -87,6 +87,7 @@
     <string name="use_fade">Use fade animations</string>
     <string name="move_backwards">Move backwards</string>
     <string name="slideshow_ended">The slideshow ended</string>
+    <string name="no_media_for_slideshow">No media for the slideshow have been found</string>
 
     <!-- Settings -->
     <string name="show_hidden_media">Show hidden media</string>

app/src/main/res/values-it/strings.xml

@@ -87,6 +87,7 @@
     <string name="use_fade">Use fade animations</string>
     <string name="move_backwards">Move backwards</string>
     <string name="slideshow_ended">The slideshow ended</string>
+    <string name="no_media_for_slideshow">No media for the slideshow have been found</string>
 
     <!-- Settings -->
     <string name="show_hidden_media">Mostra cartelle nascoste</string>

app/src/main/res/values-ja/strings.xml

@@ -87,6 +87,7 @@
     <string name="use_fade">Use fade animations</string>
     <string name="move_backwards">Move backwards</string>
     <string name="slideshow_ended">The slideshow ended</string>
+    <string name="no_media_for_slideshow">No media for the slideshow have been found</string>
 
     <!-- Settings -->
     <string name="show_hidden_media">非表示フォルダーを表示</string>

app/src/main/res/values-pl/strings.xml

@@ -87,6 +87,7 @@
     <string name="use_fade">Use fade animations</string>
     <string name="move_backwards">Move backwards</string>
     <string name="slideshow_ended">The slideshow ended</string>
+    <string name="no_media_for_slideshow">No media for the slideshow have been found</string>
 
     <!-- Settings -->
     <string name="show_hidden_media">Pokazuj ukryte foldery</string>

app/src/main/res/values-pt-rBR/strings.xml

@@ -88,6 +88,7 @@
     <string name="use_fade">Use fade animations</string>
     <string name="move_backwards">Move backwards</string>
     <string name="slideshow_ended">The slideshow ended</string>
+    <string name="no_media_for_slideshow">No media for the slideshow have been found</string>
 
     <!-- Settings -->
     <string name="show_hidden_media">Mostrar pastas ocultas</string>

app/src/main/res/values-pt/strings.xml

@@ -87,6 +87,7 @@
     <string name="use_fade">Use fade animations</string>
     <string name="move_backwards">Move backwards</string>
     <string name="slideshow_ended">The slideshow ended</string>
+    <string name="no_media_for_slideshow">No media for the slideshow have been found</string>
 
     <!-- Settings -->
     <string name="show_hidden_media">Mostrar pastas ocultas</string>

app/src/main/res/values-ru/strings.xml

@@ -87,6 +87,7 @@
     <string name="use_fade">Use fade animations</string>
     <string name="move_backwards">Move backwards</string>
     <string name="slideshow_ended">The slideshow ended</string>
+    <string name="no_media_for_slideshow">No media for the slideshow have been found</string>
 
     <!-- Settings -->
     <string name="show_hidden_media">Показать скрытые папки</string>

app/src/main/res/values-sk/strings.xml

@@ -87,6 +87,7 @@
     <string name="use_fade">Používať miznúce animácie</string>
     <string name="move_backwards">Ísť opačným smerom</string>
     <string name="slideshow_ended">Prezentácia skončila</string>
+    <string name="no_media_for_slideshow">Pre prezentáciu sa nenašli žiadne vhodné súbory</string>
 
     <!-- Settings -->
     <string name="show_hidden_media">Zobraziť skryté médiá</string>

app/src/main/res/values-sv/strings.xml

@@ -87,6 +87,7 @@
     <string name="use_fade">Use fade animations</string>
     <string name="move_backwards">Move backwards</string>
     <string name="slideshow_ended">The slideshow ended</string>
+    <string name="no_media_for_slideshow">No media for the slideshow have been found</string>
 
     <!-- Settings -->
     <string name="show_hidden_media">Visa dolda mappar</string>

app/src/main/res/values-tr/strings.xml

@@ -87,6 +87,7 @@
     <string name="use_fade">Use fade animations</string>
     <string name="move_backwards">Move backwards</string>
     <string name="slideshow_ended">The slideshow ended</string>
+    <string name="no_media_for_slideshow">No media for the slideshow have been found</string>
 
     <!-- Settings -->
     <string name="show_hidden_media">Gizli klasörleri göster</string>

app/src/main/res/values-zh-rCN/strings.xml

@@ -87,6 +87,7 @@
     <string name="use_fade">Use fade animations</string>
     <string name="move_backwards">Move backwards</string>
     <string name="slideshow_ended">The slideshow ended</string>
+    <string name="no_media_for_slideshow">No media for the slideshow have been found</string>
 
     <!-- Settings -->
     <string name="show_hidden_media">显示所有</string>

app/src/main/res/values-zh-rTW/strings.xml

@@ -87,6 +87,7 @@
     <string name="use_fade">Use fade animations</string>
     <string name="move_backwards">Move backwards</string>
     <string name="slideshow_ended">The slideshow ended</string>
+    <string name="no_media_for_slideshow">No media for the slideshow have been found</string>
 
     <!-- Settings -->
     <string name="show_hidden_media">秀出隱藏資料夾</string>

app/src/main/res/values/strings.xml

@@ -87,6 +87,7 @@
     <string name="use_fade">Use fade animations</string>
     <string name="move_backwards">Move backwards</string>
     <string name="slideshow_ended">The slideshow ended</string>
+    <string name="no_media_for_slideshow">No media for the slideshow have been found</string>
 
     <!-- Settings -->
     <string name="show_hidden_media">Show hidden media</string>