fix: Hide appBar if it's compose

Also minor i18n correction
This commit is contained in:
ziro 2024-01-15 12:08:40 +07:00
parent 5c18fb54a1
commit d8e5e78a7e
Signed by: null2264
GPG key ID: BA64F8B60AF3EFB6
8 changed files with 12 additions and 14 deletions

View file

@ -17,7 +17,6 @@ abstract class BaseComposeController(bundle: Bundle? = null) :
container: ViewGroup,
savedViewState: Bundle?
): View {
hideLegacyAppBar()
return ComposeView(container.context).apply {
layoutParams = ViewGroup.LayoutParams(
ViewGroup.LayoutParams.MATCH_PARENT,

View file

@ -141,12 +141,4 @@ abstract class BaseController(bundle: Bundle? = null) :
true
}
}
fun hideLegacyAppBar() {
(activity as? AppCompatActivity)?.findViewById<View>(R.id.app_bar)?.isVisible = false
}
fun showLegacyAppBar() {
(activity as? AppCompatActivity)?.findViewById<View>(R.id.app_bar)?.isVisible = true
}
}

View file

@ -24,7 +24,6 @@ abstract class BaseLegacyController<VB : ViewBinding>(bundle: Bundle? = null) :
val isBindingInitialized get() = this::binding.isInitialized
override fun onCreateView(inflater: LayoutInflater, container: ViewGroup, savedViewState: Bundle?): View {
showLegacyAppBar()
binding = createBinding(inflater)
binding.root.backgroundColor = binding.root.context.getResourceColor(R.attr.background)
return binding.root

View file

@ -127,6 +127,7 @@ import eu.kanade.tachiyomi.util.view.canStillGoBack
import eu.kanade.tachiyomi.util.view.doOnApplyWindowInsetsCompat
import eu.kanade.tachiyomi.util.view.findChild
import eu.kanade.tachiyomi.util.view.getItemView
import eu.kanade.tachiyomi.util.view.isCompose
import eu.kanade.tachiyomi.util.view.mainRecyclerView
import eu.kanade.tachiyomi.util.view.snack
import eu.kanade.tachiyomi.util.view.withFadeInTransaction
@ -166,6 +167,8 @@ open class MainActivity : BaseActivity<MainActivityBinding>() {
private val extensionManager: ExtensionManager by injectLazy()
private val hideBottomNav
get() = router.backstackSize > 1 && router.backstack[1].controller !is DialogController
private val hideAppBar
get() = router.isCompose
private val updateChecker by lazy { AppUpdateChecker() }
private val isUpdaterEnabled = BuildConfig.INCLUDE_UPDATER
@ -564,7 +567,7 @@ open class MainActivity : BaseActivity<MainActivityBinding>() {
) {
to?.view?.alpha = 1f
syncActivityViewWithController(to, from, isPush)
binding.appBar.isVisible = true
binding.appBar.isVisible = !hideAppBar
binding.appBar.alpha = 1f
if (binding.backShadow.isVisible && !isPush) {
val bA = ObjectAnimator.ofFloat(binding.backShadow, View.ALPHA, 0f)

View file

@ -373,7 +373,7 @@ class SettingsAdvancedController : SettingsController() {
}
}
preference {
titleRes = R.string.action_revoke_all_extension
titleRes = R.string.action_revoke_all_extensions
onClick {
trustExtension.revokeAll()

View file

@ -55,6 +55,7 @@ import eu.kanade.tachiyomi.util.view.checkHeightThen
import eu.kanade.tachiyomi.util.view.collapse
import eu.kanade.tachiyomi.util.view.expand
import eu.kanade.tachiyomi.util.view.isCollapsed
import eu.kanade.tachiyomi.util.view.isCompose
import eu.kanade.tachiyomi.util.view.isControllerVisible
import eu.kanade.tachiyomi.util.view.requestFilePermissionsSafe
import eu.kanade.tachiyomi.util.view.scrollViewWith
@ -543,7 +544,7 @@ class BrowseController :
if (!type.isEnter) {
binding.bottomSheet.root.canExpand = false
activityBinding?.appBar?.alpha = 1f
activityBinding?.appBar?.isInvisible = false
activityBinding?.appBar?.isInvisible = router.isCompose
binding.bottomSheet.sheetToolbar.menu.findItem(R.id.action_search)?.let { searchItem ->
val searchView = searchItem.actionView as SearchView
searchView.clearFocus()

View file

@ -57,6 +57,7 @@ import eu.kanade.tachiyomi.data.backup.BackupCreatorJob
import eu.kanade.tachiyomi.data.preference.PreferencesHelper
import eu.kanade.tachiyomi.databinding.MainActivityBinding
import eu.kanade.tachiyomi.ui.base.SmallToolbarInterface
import eu.kanade.tachiyomi.ui.base.controller.BaseComposeController
import eu.kanade.tachiyomi.ui.base.controller.BaseController
import eu.kanade.tachiyomi.ui.base.controller.BaseLegacyController
import eu.kanade.tachiyomi.ui.base.controller.CrossFadeChangeHandler
@ -910,6 +911,9 @@ fun Router.canStillGoBack(): Boolean {
return false
}
val Router.isCompose: Boolean
get() = backstack.lastOrNull()?.controller is BaseComposeController
interface BackHandlerControllerInterface {
fun handleOnBackStarted(backEvent: BackEventCompat) {
}

View file

@ -361,7 +361,7 @@
<item quantity="one">Extension update available</item>
<item quantity="other">%d extension updates available</item>
</plurals>
<string name="action_revoke_all_extension">Revoke all trusted extension</string>
<string name="action_revoke_all_extensions">Revoke all trusted extensions</string>
<!-- Reader -->
<string name="set_as_cover">Set as cover</string>