Fabio/Auxio
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

detail: fix bouncing when navigating to song

Browse source
This commit is contained in:
Alexander Capehart 2025-01-11 10:10:20 -07:00
parent 85a2952ae1
commit 698f0bc13c
No known key found for this signature in database
GPG key ID: 37DBE3621FE9AD47
1 changed files with 8 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
app/src/main/java/org/oxycblt/auxio/detail/AlbumDetailFragment.kt
View file

@ -22,6 +22,7 @@ import android.os.Bundle
import androidx.navigation.fragment.findNavController import androidx.navigation.fragment.findNavController
import androidx.navigation.fragment.navArgs import androidx.navigation.fragment.navArgs
import androidx.recyclerview.widget.LinearSmoothScroller import androidx.recyclerview.widget.LinearSmoothScroller
import androidx.recyclerview.widget.RecyclerView
import dagger.hilt.android.AndroidEntryPoint import dagger.hilt.android.AndroidEntryPoint
import org.oxycblt.auxio.R import org.oxycblt.auxio.R
import org.oxycblt.auxio.databinding.FragmentDetailBinding import org.oxycblt.auxio.databinding.FragmentDetailBinding
@ -298,6 +299,11 @@ class AlbumDetailFragment : DetailFragment<Album, Song>() {
// RecyclerView will scroll assuming it has the total height of the screen (i.e a // RecyclerView will scroll assuming it has the total height of the screen (i.e a
// collapsed appbar), so we need to collapse the appbar if that's the case. // collapsed appbar), so we need to collapse the appbar if that's the case.
binding.detailAppbar.setExpanded(false) binding.detailAppbar.setExpanded(false)
if (!binding.detailRecycler.canScroll()) {
// Don't scroll if the RecyclerView goes off screen. If we go anyway, overscroll
// kicks in and creates a weird bounce effect.
return
}
binding.detailRecycler.post { binding.detailRecycler.post {
// Use a custom smooth scroller that will settle the item in the middle of // Use a custom smooth scroller that will settle the item in the middle of
// the screen rather than the end. // the screen rather than the end.
@ -323,4 +329,6 @@ class AlbumDetailFragment : DetailFragment<Album, Song>() {
} }
} }
} }
private fun RecyclerView.canScroll() = computeVerticalScrollRange() > height
} }