ui: fix queue sheet not being scrollable
Fix an issue where the recent BackportBottomSheetBehavior update would break queue sheet scrolling, as the queue sheet would be set to invisible and prevent the findScrollingChild helper from being able to locate it. Fix this by removing that visibility check in the first place.
This commit is contained in:
parent
ff45380857
commit
f8fbcac482
2 changed files with 15 additions and 3 deletions
13
CHANGELOG.md
13
CHANGELOG.md
|
@ -1,5 +1,18 @@
|
||||||
# Changelog
|
# Changelog
|
||||||
|
|
||||||
|
## dev
|
||||||
|
|
||||||
|
#### What's New
|
||||||
|
- Menus have been refreshed with a cleaner look
|
||||||
|
|
||||||
|
#### What's Fixed
|
||||||
|
- Fixed issue where one could not navigate to settings after navigating
|
||||||
|
elsewhere
|
||||||
|
- Fixed the queue list being non-scrollable in certain cases
|
||||||
|
|
||||||
|
#### Dev/Meta
|
||||||
|
- Unified navigation graph
|
||||||
|
|
||||||
## 3.1.3
|
## 3.1.3
|
||||||
|
|
||||||
#### What's New
|
#### What's New
|
||||||
|
|
|
@ -1671,9 +1671,8 @@ public class BackportBottomSheetBehavior<V extends View> extends CoordinatorLayo
|
||||||
@Nullable
|
@Nullable
|
||||||
@VisibleForTesting
|
@VisibleForTesting
|
||||||
View findScrollingChild(View view) {
|
View findScrollingChild(View view) {
|
||||||
if (view.getVisibility() != View.VISIBLE) {
|
// MODIFICATION: Remove visibility check that broke nested scrolling in the queue sheet
|
||||||
return null;
|
// due to it being set to invisible when completely hidden
|
||||||
}
|
|
||||||
if (ViewCompat.isNestedScrollingEnabled(view)) {
|
if (ViewCompat.isNestedScrollingEnabled(view)) {
|
||||||
return view;
|
return view;
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue