Update seekbar visibility correctly
If seekbar is configured hidden, it should stay hidden regardless of
the visibility of control bar scrim. Currently we have a list of view
ids in the resource file. These views need to be shown/hidden based
on the scrim visibility, and seek bar is in that list. We can either
ignore seekbar at runtime (this CL does), or delete the seek bar from
the resource file. The latter is hard to mantain because it requires
to configure show_linear_progress_bar and
playback_views_to_hide_when_showing_custom_actions at the same time,
so this CL uses the former solution.
Fixes: 138647453
Test: manual
Change-Id: I0c3d97498f3bf95390ff35131b476be157aecd64
diff --git a/src/com/android/car/media/PlaybackFragment.java b/src/com/android/car/media/PlaybackFragment.java
index 92f50e1..8773795 100644
--- a/src/com/android/car/media/PlaybackFragment.java
+++ b/src/com/android/car/media/PlaybackFragment.java
@@ -330,7 +330,7 @@
int viewId = hideViewIds.getResourceId(i, 0);
if (viewId != 0) {
View viewToHide = view.findViewById(viewId);
- if (viewToHide != null) {
+ if (viewToHide != null && (mShowLinearProgressBar || viewToHide != mSeekBar)) {
mViewsToHideForCustomActions.add(viewToHide);
}
}
@@ -468,7 +468,7 @@
TextView curTime = view.findViewById(R.id.current_time);
TextView innerSeparator = view.findViewById(R.id.inner_separator);
TextView maxTime = view.findViewById(R.id.max_time);
- SeekBar seekbar = mShowLinearProgressBar? mSeekBar : null;
+ SeekBar seekbar = mShowLinearProgressBar ? mSeekBar : null;
mMetadataController = new MetadataController(getViewLifecycleOwner(),
getPlaybackViewModel(), title, artist, albumTitle, outerSeparator,
