kavin/LibreTube
1
0
Fork 0
mirror of https://github.com/libre-tube/LibreTube.git synced 2024-12-14 06:10:31 +05:30
Code Issues Projects Releases Wiki Activity

Merge pull request #800 from Bnyro/master

Browse Source 
small cleanup
This commit is contained in:
Bnyro 2022-07-16 20:59:15 +02:00 committed by GitHub
commit a50bf8d00b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 15 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
app/src/main/java/com/github/libretube/Globals.kt
View File

@ -8,9 +8,6 @@ object Globals {
var isFullScreen = false
var isMiniPlayerVisible = false
// for the settings
var isCurrentViewMainSettings = true
// for the data saver mode
var dataSaverModeEnabled = false
}

21
app/src/main/java/com/github/libretube/fragments/PlayerFragment.kt
View File

@ -176,7 +176,7 @@ class PlayerFragment : Fragment() {
val mainActivity = activity as MainActivity
if (autoRotationEnabled) {
// enable auto rotation
mainActivity.requestedOrientation = ActivityInfo.SCREEN_ORIENTATION_USER
mainActivity.requestedOrientation = ActivityInfo.SCREEN_ORIENTATION_SENSOR
onConfigurationChanged(resources.configuration)
} else {
// go to portrait mode
@ -376,6 +376,7 @@ class PlayerFragment : Fragment() {
binding.linLayout.visibility = View.GONE
playerBinding.fullscreen.setImageResource(R.drawable.ic_fullscreen_exit)
playerBinding.exoTitle.visibility = View.VISIBLE
playerBinding.closeImageButton.visibility = View.GONE
val mainActivity = activity as MainActivity
val fullscreenOrientationPref = PreferenceHelper
@ -393,7 +394,7 @@ class PlayerFragment : Fragment() {
// a video with normal aspect ratio
else ActivityInfo.SCREEN_ORIENTATION_USER_LANDSCAPE
}
"auto" -> ActivityInfo.SCREEN_ORIENTATION_USER
"auto" -> ActivityInfo.SCREEN_ORIENTATION_SENSOR
"landscape" -> ActivityInfo.SCREEN_ORIENTATION_USER_LANDSCAPE
"portrait" -> ActivityInfo.SCREEN_ORIENTATION_USER_PORTRAIT
else -> ActivityInfo.SCREEN_ORIENTATION_USER_LANDSCAPE
@ -415,6 +416,7 @@ class PlayerFragment : Fragment() {
binding.linLayout.visibility = View.VISIBLE
playerBinding.fullscreen.setImageResource(R.drawable.ic_fullscreen)
playerBinding.exoTitle.visibility = View.INVISIBLE
playerBinding.closeImageButton.visibility = View.VISIBLE
scaleControls(1F)
@ -1320,7 +1322,16 @@ class PlayerFragment : Fragment() {
playerBinding.exoBottomBar.visibility = visibility
playerBinding.closeImageButton.visibility = visibility
playerBinding.exoTitle.visibility =
if (isLocked && Globals.isFullScreen) View.VISIBLE else View.INVISIBLE
if (isLocked &&
Globals.isFullScreen
) View.VISIBLE else View.INVISIBLE
// hide the close image button
playerBinding.closeImageButton.visibility =
if (isLocked &&
!Globals.isFullScreen &&
autoRotationEnabled
) View.VISIBLE else View.GONE
// disable double tap to seek when the player is locked
if (isLocked) enableDoubleTapToSeek() else disableDoubleTapToSeek()
@ -1481,15 +1492,13 @@ class PlayerFragment : Fragment() {
exoPlayerView.hideController()
exoPlayerView.useController = false
// set portrait mode
unsetFullscreen()
Globals.isFullScreen = false
} else {
// enable exoPlayer controls again
exoPlayerView.useController = true
// switch back to portrait mode
unsetFullscreen()
}
}

2
app/src/main/java/com/github/libretube/preferences/MainSettings.kt
View File

@ -7,7 +7,6 @@ import androidx.preference.ListPreference
import androidx.preference.Preference
import androidx.preference.PreferenceFragmentCompat
import com.github.libretube.BuildConfig
import com.github.libretube.Globals
import com.github.libretube.R
import com.github.libretube.dialogs.RequireRestartDialog
import com.github.libretube.util.ThemeHelper
@ -94,7 +93,6 @@ class MainSettings : PreferenceFragmentCompat() {
}
private fun navigateToSettingsFragment(newFragment: Fragment) {
Globals.isCurrentViewMainSettings = false
parentFragmentManager.beginTransaction()
.replace(R.id.settings, newFragment)
.commitNow()