From 040d3e4433c2ab3d425846ecc5d66b606a3fffd3 Mon Sep 17 00:00:00 2001 From: Koitharu Date: Fri, 26 Aug 2022 12:08:19 +0300 Subject: [PATCH] Reader control direction depends on mode #214 --- .../kotatsu/core/prefs/AppSettings.kt | 7 +- .../main/ui/protect/AppProtectHelper.kt | 3 +- .../kotatsu/reader/ui/ReaderActivity.kt | 13 ++-- .../reader/ui/ReaderControlDelegate.kt | 67 +++++++++++++------ app/src/main/res/values/strings.xml | 4 +- app/src/main/res/xml/pref_reader.xml | 6 ++ 6 files changed, 70 insertions(+), 30 deletions(-) diff --git a/app/src/main/java/org/koitharu/kotatsu/core/prefs/AppSettings.kt b/app/src/main/java/org/koitharu/kotatsu/core/prefs/AppSettings.kt index cd9c8a07a..2500fe656 100644 --- a/app/src/main/java/org/koitharu/kotatsu/core/prefs/AppSettings.kt +++ b/app/src/main/java/org/koitharu/kotatsu/core/prefs/AppSettings.kt @@ -14,9 +14,6 @@ import java.io.File import java.text.DateFormat import java.text.SimpleDateFormat import java.util.* -import kotlinx.coroutines.channels.awaitClose -import kotlinx.coroutines.channels.trySendBlocking -import kotlinx.coroutines.flow.callbackFlow import org.koitharu.kotatsu.BuildConfig import org.koitharu.kotatsu.core.model.ZoomMode import org.koitharu.kotatsu.core.network.DoHProvider @@ -64,6 +61,9 @@ class AppSettings(context: Context) { val readerPageSwitch: Set get() = prefs.getStringSet(KEY_READER_SWITCHERS, null) ?: setOf(PAGE_SWITCH_TAPS) + val isReaderTapsAdaptive: Boolean + get() = !prefs.getBoolean(KEY_READER_TAPS_LTR, false) + var isTrafficWarningEnabled: Boolean get() = prefs.getBoolean(KEY_TRAFFIC_WARNING, true) set(value) = prefs.edit { putBoolean(KEY_TRAFFIC_WARNING, value) } @@ -314,6 +314,7 @@ class AppSettings(context: Context) { const val KEY_DOWNLOADS_SLOWDOWN = "downloads_slowdown" const val KEY_ALL_FAVOURITES_VISIBLE = "all_favourites_visible" const val KEY_DOH = "doh" + const val KEY_READER_TAPS_LTR = "reader_taps_ltr" // About const val KEY_APP_UPDATE = "app_update" diff --git a/app/src/main/java/org/koitharu/kotatsu/main/ui/protect/AppProtectHelper.kt b/app/src/main/java/org/koitharu/kotatsu/main/ui/protect/AppProtectHelper.kt index a92d866d2..45f7e9ea4 100644 --- a/app/src/main/java/org/koitharu/kotatsu/main/ui/protect/AppProtectHelper.kt +++ b/app/src/main/java/org/koitharu/kotatsu/main/ui/protect/AppProtectHelper.kt @@ -4,6 +4,7 @@ import android.app.Activity import android.app.Application import android.content.Intent import android.os.Bundle +import org.acra.dialog.CrashReportDialog import org.koitharu.kotatsu.core.prefs.AppSettings class AppProtectHelper(private val settings: AppSettings) : Application.ActivityLifecycleCallbacks { @@ -11,7 +12,7 @@ class AppProtectHelper(private val settings: AppSettings) : Application.Activity private var isUnlocked = settings.appPassword.isNullOrEmpty() override fun onActivityCreated(activity: Activity, savedInstanceState: Bundle?) { - if (activity !is ProtectActivity && !isUnlocked) { + if (!isUnlocked && activity !is ProtectActivity && activity !is CrashReportDialog) { val sourceIntent = Intent(activity, activity.javaClass) activity.intent?.let { sourceIntent.putExtras(it) diff --git a/app/src/main/java/org/koitharu/kotatsu/reader/ui/ReaderActivity.kt b/app/src/main/java/org/koitharu/kotatsu/reader/ui/ReaderActivity.kt index 35d203b26..2bf5c641a 100644 --- a/app/src/main/java/org/koitharu/kotatsu/reader/ui/ReaderActivity.kt +++ b/app/src/main/java/org/koitharu/kotatsu/reader/ui/ReaderActivity.kt @@ -19,6 +19,7 @@ import androidx.transition.TransitionManager import androidx.transition.TransitionSet import com.google.android.material.dialog.MaterialAlertDialogBuilder import com.google.android.material.snackbar.Snackbar +import java.util.concurrent.TimeUnit import kotlinx.coroutines.Dispatchers import kotlinx.coroutines.flow.launchIn import kotlinx.coroutines.flow.onEach @@ -47,7 +48,6 @@ import org.koitharu.kotatsu.utils.GridTouchHelper import org.koitharu.kotatsu.utils.ScreenOrientationHelper import org.koitharu.kotatsu.utils.ShareHelper import org.koitharu.kotatsu.utils.ext.* -import java.util.concurrent.TimeUnit class ReaderActivity : BaseFullscreenActivity(), @@ -67,6 +67,9 @@ class ReaderActivity : ) } + override val readerMode: ReaderMode? + get() = readerManager.currentMode + private lateinit var touchHelper: GridTouchHelper private lateinit var orientationHelper: ScreenOrientationHelper private lateinit var controlDelegate: ReaderControlDelegate @@ -82,7 +85,7 @@ class ReaderActivity : supportActionBar?.setDisplayHomeAsUpEnabled(true) touchHelper = GridTouchHelper(this, this) orientationHelper = ScreenOrientationHelper(this) - controlDelegate = ReaderControlDelegate(lifecycleScope, get(), this) + controlDelegate = ReaderControlDelegate(get(), this, this) binding.toolbarBottom.inflateMenu(R.menu.opt_reader_bottom) binding.toolbarBottom.setOnMenuItemClickListener(::onOptionsItemSelected) insetsDelegate.interceptingWindowInsetsListener = this @@ -146,7 +149,7 @@ class ReaderActivity : ChaptersBottomSheet.show( supportFragmentManager, viewModel.manga?.chapters.orEmpty(), - viewModel.getCurrentState()?.chapterId ?: 0L + viewModel.getCurrentState()?.chapterId ?: 0L, ) } R.id.action_screen_rotate -> { @@ -317,12 +320,12 @@ class ReaderActivity : binding.appbarTop.updatePadding( top = systemBars.top, right = systemBars.right, - left = systemBars.left + left = systemBars.left, ) binding.appbarBottom?.updatePadding( bottom = systemBars.bottom, right = systemBars.right, - left = systemBars.left + left = systemBars.left, ) return WindowInsetsCompat.Builder(insets) .setInsets(WindowInsetsCompat.Type.systemBars(), Insets.NONE) diff --git a/app/src/main/java/org/koitharu/kotatsu/reader/ui/ReaderControlDelegate.kt b/app/src/main/java/org/koitharu/kotatsu/reader/ui/ReaderControlDelegate.kt index dbe853894..6fbf45e28 100644 --- a/app/src/main/java/org/koitharu/kotatsu/reader/ui/ReaderControlDelegate.kt +++ b/app/src/main/java/org/koitharu/kotatsu/reader/ui/ReaderControlDelegate.kt @@ -1,33 +1,39 @@ package org.koitharu.kotatsu.reader.ui +import android.content.SharedPreferences import android.view.KeyEvent import android.view.SoundEffectConstants import android.view.View -import androidx.lifecycle.LifecycleCoroutineScope -import kotlinx.coroutines.Dispatchers -import kotlinx.coroutines.flow.flowOn -import kotlinx.coroutines.flow.launchIn -import kotlinx.coroutines.flow.onEach +import androidx.lifecycle.DefaultLifecycleObserver +import androidx.lifecycle.LifecycleOwner import org.koitharu.kotatsu.core.prefs.AppSettings -import org.koitharu.kotatsu.core.prefs.observeAsFlow +import org.koitharu.kotatsu.core.prefs.ReaderMode import org.koitharu.kotatsu.utils.GridTouchHelper class ReaderControlDelegate( - scope: LifecycleCoroutineScope, - settings: AppSettings, - private val listener: OnInteractionListener -) { + private val settings: AppSettings, + private val listener: OnInteractionListener, + owner: LifecycleOwner, +) : DefaultLifecycleObserver, SharedPreferences.OnSharedPreferenceChangeListener { private var isTapSwitchEnabled: Boolean = true private var isVolumeKeysSwitchEnabled: Boolean = false + private var isReaderTapsAdaptive: Boolean = true init { - settings.observeAsFlow(AppSettings.KEY_READER_SWITCHERS) { readerPageSwitch } - .flowOn(Dispatchers.Default) - .onEach { - isTapSwitchEnabled = AppSettings.PAGE_SWITCH_TAPS in it - isVolumeKeysSwitchEnabled = AppSettings.PAGE_SWITCH_VOLUME_KEYS in it - }.launchIn(scope) + owner.lifecycle.addObserver(this) + settings.subscribe(this) + updateSettings() + } + + override fun onDestroy(owner: LifecycleOwner) { + settings.unsubscribe(this) + owner.lifecycle.removeObserver(this) + super.onDestroy(owner) + } + + override fun onSharedPreferenceChanged(sharedPreferences: SharedPreferences?, key: String?) { + updateSettings() } fun onGridTouch(area: Int, view: View) { @@ -41,7 +47,7 @@ class ReaderControlDelegate( view.playSoundEffect(SoundEffectConstants.NAVIGATION_UP) } GridTouchHelper.AREA_LEFT -> if (isTapSwitchEnabled) { - listener.switchPageBy(-1) + listener.switchPageBy(if (isReaderTapsReversed()) 1 else -1) view.playSoundEffect(SoundEffectConstants.NAVIGATION_LEFT) } GridTouchHelper.AREA_BOTTOM -> if (isTapSwitchEnabled) { @@ -49,7 +55,7 @@ class ReaderControlDelegate( view.playSoundEffect(SoundEffectConstants.NAVIGATION_DOWN) } GridTouchHelper.AREA_RIGHT -> if (isTapSwitchEnabled) { - listener.switchPageBy(1) + listener.switchPageBy(if (isReaderTapsReversed()) -1 else 1) view.playSoundEffect(SoundEffectConstants.NAVIGATION_RIGHT) } } @@ -72,17 +78,25 @@ class ReaderControlDelegate( KeyEvent.KEYCODE_PAGE_DOWN, KeyEvent.KEYCODE_SYSTEM_NAVIGATION_DOWN, KeyEvent.KEYCODE_DPAD_DOWN, - KeyEvent.KEYCODE_DPAD_RIGHT -> { + -> { listener.switchPageBy(1) true } + KeyEvent.KEYCODE_DPAD_RIGHT -> { + listener.switchPageBy(if (isReaderTapsReversed()) -1 else 1) + true + } KeyEvent.KEYCODE_PAGE_UP, KeyEvent.KEYCODE_SYSTEM_NAVIGATION_UP, KeyEvent.KEYCODE_DPAD_UP, - KeyEvent.KEYCODE_DPAD_LEFT -> { + -> { listener.switchPageBy(-1) true } + KeyEvent.KEYCODE_DPAD_LEFT -> { + listener.switchPageBy(if (isReaderTapsReversed()) 1 else -1) + true + } KeyEvent.KEYCODE_DPAD_CENTER -> { listener.toggleUiVisibility() true @@ -97,8 +111,21 @@ class ReaderControlDelegate( ) } + private fun updateSettings() { + val switch = settings.readerPageSwitch + isTapSwitchEnabled = AppSettings.PAGE_SWITCH_TAPS in switch + isVolumeKeysSwitchEnabled = AppSettings.PAGE_SWITCH_VOLUME_KEYS in switch + isReaderTapsAdaptive = settings.isReaderTapsAdaptive + } + + private fun isReaderTapsReversed(): Boolean { + return isReaderTapsAdaptive && listener.readerMode == ReaderMode.REVERSED + } + interface OnInteractionListener { + val readerMode: ReaderMode? + fun switchPageBy(delta: Int) fun toggleUiVisibility() diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml index f33ed79f9..fb55d5185 100644 --- a/app/src/main/res/values/strings.xml +++ b/app/src/main/res/values/strings.xml @@ -325,4 +325,6 @@ Content not found or removed Downloading manga <b>%1$s</b> %2$s - \ No newline at end of file + Tap on the right edge or pressing the right key always switches to the next page + Ergonomic reader control + diff --git a/app/src/main/res/xml/pref_reader.xml b/app/src/main/res/xml/pref_reader.xml index 42e46b8ad..d8c4fe0c3 100644 --- a/app/src/main/res/xml/pref_reader.xml +++ b/app/src/main/res/xml/pref_reader.xml @@ -29,6 +29,12 @@ android:title="@string/switch_pages" app:allowDividerAbove="true" /> + +