always include photos in slideshows
This commit is contained in:
parent
62cbdaf918
commit
e464926a5c
5 changed files with 2 additions and 41 deletions
|
@ -505,7 +505,7 @@ class ViewPagerActivity : SimpleActivity(), ViewPager.OnPageChangeListener, View
|
||||||
|
|
||||||
private fun getMediaForSlideshow(): Boolean {
|
private fun getMediaForSlideshow(): Boolean {
|
||||||
mSlideshowMedia = mMediaFiles.filter {
|
mSlideshowMedia = mMediaFiles.filter {
|
||||||
(config.slideshowIncludePhotos && it.isImage()) || (config.slideshowIncludeGIFs && it.isGIF())
|
it.isImage() || (config.slideshowIncludeGIFs && it.isGIF())
|
||||||
}.toMutableList()
|
}.toMutableList()
|
||||||
|
|
||||||
if (config.slideshowRandomOrder) {
|
if (config.slideshowRandomOrder) {
|
||||||
|
|
|
@ -5,7 +5,6 @@ import androidx.appcompat.app.AlertDialog
|
||||||
import com.simplemobiletools.commons.activities.BaseSimpleActivity
|
import com.simplemobiletools.commons.activities.BaseSimpleActivity
|
||||||
import com.simplemobiletools.commons.extensions.hideKeyboard
|
import com.simplemobiletools.commons.extensions.hideKeyboard
|
||||||
import com.simplemobiletools.commons.extensions.setupDialogStuff
|
import com.simplemobiletools.commons.extensions.setupDialogStuff
|
||||||
import com.simplemobiletools.commons.extensions.toast
|
|
||||||
import com.simplemobiletools.gallery.pro.R
|
import com.simplemobiletools.gallery.pro.R
|
||||||
import com.simplemobiletools.gallery.pro.extensions.config
|
import com.simplemobiletools.gallery.pro.extensions.config
|
||||||
import com.simplemobiletools.gallery.pro.helpers.SLIDESHOW_DEFAULT_INTERVAL
|
import com.simplemobiletools.gallery.pro.helpers.SLIDESHOW_DEFAULT_INTERVAL
|
||||||
|
@ -29,11 +28,6 @@ class SlideshowDialog(val activity: BaseSimpleActivity, val callback: () -> Unit
|
||||||
activity.hideKeyboard(v)
|
activity.hideKeyboard(v)
|
||||||
}
|
}
|
||||||
|
|
||||||
include_photos_holder.setOnClickListener {
|
|
||||||
interval_value.clearFocus()
|
|
||||||
include_photos.toggle()
|
|
||||||
}
|
|
||||||
|
|
||||||
include_gifs_holder.setOnClickListener {
|
include_gifs_holder.setOnClickListener {
|
||||||
interval_value.clearFocus()
|
interval_value.clearFocus()
|
||||||
include_gifs.toggle()
|
include_gifs.toggle()
|
||||||
|
@ -68,11 +62,6 @@ class SlideshowDialog(val activity: BaseSimpleActivity, val callback: () -> Unit
|
||||||
activity.setupDialogStuff(view, this) {
|
activity.setupDialogStuff(view, this) {
|
||||||
hideKeyboard()
|
hideKeyboard()
|
||||||
getButton(AlertDialog.BUTTON_POSITIVE).setOnClickListener {
|
getButton(AlertDialog.BUTTON_POSITIVE).setOnClickListener {
|
||||||
if (!view.include_photos.isChecked && !view.include_gifs.isChecked) {
|
|
||||||
activity.toast(R.string.no_media_for_slideshow)
|
|
||||||
return@setOnClickListener
|
|
||||||
}
|
|
||||||
|
|
||||||
storeValues()
|
storeValues()
|
||||||
callback()
|
callback()
|
||||||
dismiss()
|
dismiss()
|
||||||
|
@ -85,7 +74,6 @@ class SlideshowDialog(val activity: BaseSimpleActivity, val callback: () -> Unit
|
||||||
val config = activity.config
|
val config = activity.config
|
||||||
view.apply {
|
view.apply {
|
||||||
interval_value.setText(config.slideshowInterval.toString())
|
interval_value.setText(config.slideshowInterval.toString())
|
||||||
include_photos.isChecked = config.slideshowIncludePhotos
|
|
||||||
include_gifs.isChecked = config.slideshowIncludeGIFs
|
include_gifs.isChecked = config.slideshowIncludeGIFs
|
||||||
random_order.isChecked = config.slideshowRandomOrder
|
random_order.isChecked = config.slideshowRandomOrder
|
||||||
use_fade.isChecked = config.slideshowUseFade
|
use_fade.isChecked = config.slideshowUseFade
|
||||||
|
@ -101,7 +89,6 @@ class SlideshowDialog(val activity: BaseSimpleActivity, val callback: () -> Unit
|
||||||
|
|
||||||
activity.config.apply {
|
activity.config.apply {
|
||||||
slideshowInterval = interval.toInt()
|
slideshowInterval = interval.toInt()
|
||||||
slideshowIncludePhotos = view.include_photos.isChecked
|
|
||||||
slideshowIncludeGIFs = view.include_gifs.isChecked
|
slideshowIncludeGIFs = view.include_gifs.isChecked
|
||||||
slideshowRandomOrder = view.random_order.isChecked
|
slideshowRandomOrder = view.random_order.isChecked
|
||||||
slideshowUseFade = view.use_fade.isChecked
|
slideshowUseFade = view.use_fade.isChecked
|
||||||
|
|
|
@ -277,10 +277,6 @@ class Config(context: Context) : BaseConfig(context) {
|
||||||
get() = prefs.getInt(SLIDESHOW_INTERVAL, SLIDESHOW_DEFAULT_INTERVAL)
|
get() = prefs.getInt(SLIDESHOW_INTERVAL, SLIDESHOW_DEFAULT_INTERVAL)
|
||||||
set(slideshowInterval) = prefs.edit().putInt(SLIDESHOW_INTERVAL, slideshowInterval).apply()
|
set(slideshowInterval) = prefs.edit().putInt(SLIDESHOW_INTERVAL, slideshowInterval).apply()
|
||||||
|
|
||||||
var slideshowIncludePhotos: Boolean
|
|
||||||
get() = prefs.getBoolean(SLIDESHOW_INCLUDE_PHOTOS, true)
|
|
||||||
set(slideshowIncludePhotos) = prefs.edit().putBoolean(SLIDESHOW_INCLUDE_PHOTOS, slideshowIncludePhotos).apply()
|
|
||||||
|
|
||||||
var slideshowIncludeGIFs: Boolean
|
var slideshowIncludeGIFs: Boolean
|
||||||
get() = prefs.getBoolean(SLIDESHOW_INCLUDE_GIFS, false)
|
get() = prefs.getBoolean(SLIDESHOW_INCLUDE_GIFS, false)
|
||||||
set(slideshowIncludeGIFs) = prefs.edit().putBoolean(SLIDESHOW_INCLUDE_GIFS, slideshowIncludeGIFs).apply()
|
set(slideshowIncludeGIFs) = prefs.edit().putBoolean(SLIDESHOW_INCLUDE_GIFS, slideshowIncludeGIFs).apply()
|
||||||
|
|
|
@ -76,7 +76,6 @@ const val SHOW_WIDGET_FOLDER_NAME = "show_widget_folder_name"
|
||||||
|
|
||||||
// slideshow
|
// slideshow
|
||||||
const val SLIDESHOW_INTERVAL = "slideshow_interval"
|
const val SLIDESHOW_INTERVAL = "slideshow_interval"
|
||||||
const val SLIDESHOW_INCLUDE_PHOTOS = "slideshow_include_photos"
|
|
||||||
const val SLIDESHOW_INCLUDE_GIFS = "slideshow_include_gifs"
|
const val SLIDESHOW_INCLUDE_GIFS = "slideshow_include_gifs"
|
||||||
const val SLIDESHOW_RANDOM_ORDER = "slideshow_random_order"
|
const val SLIDESHOW_RANDOM_ORDER = "slideshow_random_order"
|
||||||
const val SLIDESHOW_USE_FADE = "slideshow_use_fade"
|
const val SLIDESHOW_USE_FADE = "slideshow_use_fade"
|
||||||
|
|
|
@ -36,32 +36,11 @@
|
||||||
android:textCursorDrawable="@null"
|
android:textCursorDrawable="@null"
|
||||||
android:textSize="@dimen/normal_text_size"/>
|
android:textSize="@dimen/normal_text_size"/>
|
||||||
|
|
||||||
<RelativeLayout
|
|
||||||
android:id="@+id/include_photos_holder"
|
|
||||||
android:layout_width="match_parent"
|
|
||||||
android:layout_height="wrap_content"
|
|
||||||
android:layout_below="@+id/interval_label"
|
|
||||||
android:background="?attr/selectableItemBackground"
|
|
||||||
android:paddingStart="@dimen/medium_margin"
|
|
||||||
android:paddingLeft="@dimen/medium_margin"
|
|
||||||
android:paddingTop="@dimen/activity_margin"
|
|
||||||
android:paddingBottom="@dimen/activity_margin">
|
|
||||||
|
|
||||||
<com.simplemobiletools.commons.views.MySwitchCompat
|
|
||||||
android:id="@+id/include_photos"
|
|
||||||
android:layout_width="match_parent"
|
|
||||||
android:layout_height="wrap_content"
|
|
||||||
android:background="@null"
|
|
||||||
android:clickable="false"
|
|
||||||
android:text="@string/include_photos"/>
|
|
||||||
|
|
||||||
</RelativeLayout>
|
|
||||||
|
|
||||||
<RelativeLayout
|
<RelativeLayout
|
||||||
android:id="@+id/include_gifs_holder"
|
android:id="@+id/include_gifs_holder"
|
||||||
android:layout_width="match_parent"
|
android:layout_width="match_parent"
|
||||||
android:layout_height="wrap_content"
|
android:layout_height="wrap_content"
|
||||||
android:layout_below="@+id/include_photos_holder"
|
android:layout_below="@+id/interval_label"
|
||||||
android:background="?attr/selectableItemBackground"
|
android:background="?attr/selectableItemBackground"
|
||||||
android:paddingStart="@dimen/medium_margin"
|
android:paddingStart="@dimen/medium_margin"
|
||||||
android:paddingLeft="@dimen/medium_margin"
|
android:paddingLeft="@dimen/medium_margin"
|
||||||
|
|
Loading…
Reference in a new issue