Fix pages saving

app/src/main/kotlin/org/koitharu/kotatsu/reader/ui/ReaderActivity.kt

@@ -14,6 +14,7 @@ import android.view.MotionEvent
 import android.view.View
 import android.view.ViewGroup.MarginLayoutParams
 import android.view.WindowManager
+import androidx.activity.result.ActivityResultCallback
 import androidx.activity.viewModels
 import androidx.core.graphics.Insets
 import androidx.core.view.OnApplyWindowInsetsListener
@@ -74,6 +75,7 @@ class ReaderActivity :
 	ReaderControlDelegate.OnInteractionListener,
 	OnApplyWindowInsetsListener,
 	IdlingDetector.Callback,
+	ActivityResultCallback<Uri?>,
 	ZoomControl.ZoomControlListener {
 
 	@Inject
@@ -83,6 +85,7 @@ class ReaderActivity :
 	lateinit var tapGridSettings: TapGridSettings
 
 	private val idlingDetector = IdlingDetector(TimeUnit.SECONDS.toMillis(10), this)
+	private val savePageRequest = registerForActivityResult(PageSaveContract(), this)
 
 	private val viewModel: ReaderViewModel by viewModels()
 
@@ -158,6 +161,10 @@ class ReaderActivity :
 		viewBinding.toolbarBottom.addMenuProvider(ReaderBottomMenuProvider(this, readerManager, viewModel))
 	}
 
+	override fun onActivityResult(result: Uri?) {
+		viewModel.onActivityResult(result)
+	}
+
 	override fun getParentActivityIntent(): Intent? {
 		val manga = viewModel.manga?.toManga() ?: return null
 		return DetailsActivity.newIntent(this, manga)
@@ -371,6 +378,11 @@ class ReaderActivity :
 		return reader.isResumed && supportFragmentManager.fragments.lastOrNull() === reader
 	}
 
+	override fun onSavePageClick() {
+		val page = viewModel.getCurrentPage() ?: return
+		viewModel.saveCurrentPage(page, savePageRequest)
+	}
+
 	private fun onReaderBarChanged(isBarEnabled: Boolean) {
 		viewBinding.infoBar.isVisible = isBarEnabled && viewBinding.appbarTop.isGone
 	}

app/src/main/kotlin/org/koitharu/kotatsu/reader/ui/config/ReaderConfigSheet.kt

@@ -39,14 +39,12 @@ import javax.inject.Inject
 @AndroidEntryPoint
 class ReaderConfigSheet :
 	BaseAdaptiveSheet<SheetReaderConfigBinding>(),
-	ActivityResultCallback<Uri?>,
 	View.OnClickListener,
 	MaterialButtonToggleGroup.OnButtonCheckedListener,
 	Slider.OnChangeListener,
 	CompoundButton.OnCheckedChangeListener {
 
 	private val viewModel by activityViewModels<ReaderViewModel>()
-	private val savePageRequest = registerForActivityResult(PageSaveContract(), this)
 
 	@Inject
 	lateinit var orientationHelper: ScreenOrientationHelper
@@ -115,8 +113,7 @@ class ReaderConfigSheet :
 			}
 
 			R.id.button_save_page -> {
-				val page = viewModel.getCurrentPage() ?: return
-				viewModel.saveCurrentPage(page, savePageRequest)
+				findCallback()?.onSavePageClick() ?: return
 				dismissAllowingStateLoss()
 			}
 
@@ -181,11 +178,6 @@ class ReaderConfigSheet :
 		(viewBinding ?: return).labelTimerValue.text = getString(R.string.speed_value, value * 10f)
 	}
 
-	override fun onActivityResult(result: Uri?) {
-		viewModel.onActivityResult(result)
-		dismissAllowingStateLoss()
-	}
-
 	private fun observeScreenOrientation() {
 		orientationHelper.observeAutoOrientation()
 			.onEach {
@@ -215,6 +207,8 @@ class ReaderConfigSheet :
 		fun onReaderModeChanged(mode: ReaderMode)
 
 		fun onDoubleModeChanged(isEnabled: Boolean)
+
+		fun onSavePageClick()
 	}
 
 	companion object {