[Android] Fix about view not scrollable
Geoffrey Métais
git at videolan.org
Tue Oct 1 14:44:38 CEST 2019
vlc-android | branch: 3.2.x | Geoffrey Métais <geoffrey.metais at gmail.com> | Fri Sep 27 16:12:41 2019 +0200| [215847a48eac57cc702ecd0680dd0676a2577589] | committer: Geoffrey Métais
Fix about view not scrollable
Fix #1049
(cherry picked from commit 0bbe0ec5f19494d30a5eae95d846957385870cc5)
> https://code.videolan.org/videolan/vlc-android/commit/215847a48eac57cc702ecd0680dd0676a2577589
---
vlc-android/res/layout/about_main.xml | 4 ++--
vlc-android/src/org/videolan/vlc/gui/AboutFragment.kt | 4 ++--
vlc-android/src/org/videolan/vlc/gui/SecondaryActivity.kt | 11 ++++++-----
3 files changed, 10 insertions(+), 9 deletions(-)
diff --git a/vlc-android/res/layout/about_main.xml b/vlc-android/res/layout/about_main.xml
index ce124d33b..313aee320 100644
--- a/vlc-android/res/layout/about_main.xml
+++ b/vlc-android/res/layout/about_main.xml
@@ -2,7 +2,7 @@
<merge xmlns:android="http://schemas.android.com/apk/res/android"
android:layout_width="match_parent"
android:layout_height="match_parent">
- <ScrollView
+ <androidx.core.widget.NestedScrollView
android:id="@+id/about_main"
android:layout_width="match_parent"
android:layout_height="match_parent">
@@ -120,7 +120,7 @@
android:textSize="12sp" />
</LinearLayout>
- </ScrollView>
+ </androidx.core.widget.NestedScrollView>
<include layout="@layout/tv_time" />
</merge>
diff --git a/vlc-android/src/org/videolan/vlc/gui/AboutFragment.kt b/vlc-android/src/org/videolan/vlc/gui/AboutFragment.kt
index ed48d6bf0..37222f3f9 100644
--- a/vlc-android/src/org/videolan/vlc/gui/AboutFragment.kt
+++ b/vlc-android/src/org/videolan/vlc/gui/AboutFragment.kt
@@ -29,8 +29,8 @@ import android.view.View
import android.view.ViewGroup
import android.webkit.WebView
import android.webkit.WebViewClient
-import android.widget.ScrollView
import androidx.appcompat.app.AppCompatActivity
+import androidx.core.widget.NestedScrollView
import androidx.fragment.app.Fragment
import androidx.viewpager.widget.ViewPager
import com.google.android.material.tabs.TabLayout
@@ -58,7 +58,7 @@ class AboutFragment : Fragment() {
(activity as? AppCompatActivity)?.supportActionBar?.title = "VLC ${BuildConfig.VERSION_NAME}"
- val aboutMain = view.findViewById<ScrollView>(org.videolan.vlc.R.id.about_main)
+ val aboutMain = view.findViewById<NestedScrollView>(org.videolan.vlc.R.id.about_main)
val webView = view.findViewById<WebView>(org.videolan.vlc.R.id.webview)
val revision = getString(org.videolan.vlc.R.string.build_revision)
diff --git a/vlc-android/src/org/videolan/vlc/gui/SecondaryActivity.kt b/vlc-android/src/org/videolan/vlc/gui/SecondaryActivity.kt
index 250e601de..a79353363 100644
--- a/vlc-android/src/org/videolan/vlc/gui/SecondaryActivity.kt
+++ b/vlc-android/src/org/videolan/vlc/gui/SecondaryActivity.kt
@@ -128,15 +128,16 @@ class SecondaryActivity : ContentActivity() {
private fun fetchSecondaryFragment(id: String) {
when (id) {
ALBUMS_SONGS -> {
- fragment = AudioAlbumsSongsFragment()
- val args = Bundle()
- args.putParcelable(AudioBrowserFragment.TAG_ITEM, intent.getParcelableExtra<Parcelable>(AudioBrowserFragment.TAG_ITEM))
- fragment!!.arguments = args
+ fragment = AudioAlbumsSongsFragment().apply {
+ val args = Bundle(1)
+ args.putParcelable(AudioBrowserFragment.TAG_ITEM, intent.getParcelableExtra<Parcelable>(AudioBrowserFragment.TAG_ITEM))
+ arguments = args
+ }
}
ABOUT -> fragment = AboutFragment()
VIDEO_GROUP_LIST -> {
fragment = VideoGridFragment().apply {
- arguments = Bundle(1).apply {
+ arguments = Bundle(2).apply {
putParcelable(KEY_FOLDER, intent.getParcelableExtra<Parcelable>(KEY_FOLDER))
putParcelable(KEY_GROUP, intent.getParcelableExtra<Parcelable>(KEY_GROUP))
}
More information about the Android
mailing list