Android ViewPager с RecyclerView неправильно работает внутри BottomSheet

Когда я пытаюсь прокрутить список, иногда это работает некорректно - BottomSheet перехватывает событие прокрутки и прячется.

Как воспроизвести это:

  1. Открыть нижний лист
  2. Изменить страницу ViewPager
  3. Попробуйте прокрутить список

Результат: BottomSheet будет скрыт.

Вот пример кода:

скомпилировать 'com.android.support:design:23.4.0'

MainActivity.java

package com.nkdroid.bottomsheetsample;

import android.os.Bundle;
import android.support.design.widget.BottomSheetBehavior;
import android.support.design.widget.TabLayout;
import android.support.v4.view.PagerAdapter;
import android.support.v4.view.ViewPager;
import android.support.v7.app.AppCompatActivity;
import android.support.v7.widget.LinearLayoutManager;
import android.support.v7.widget.RecyclerView;
import android.view.View;
import android.view.ViewGroup;
import android.widget.Button;
import android.widget.TextView;

public
class MainActivity
        extends AppCompatActivity
{

    private BottomSheetBehavior behavior;

    @Override
    protected
    void onCreate(final Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        final Button btnView = (Button) findViewById(R.id.btnView);
        btnView.setOnClickListener(new View.OnClickListener()
        {
            @Override
            public
            void onClick(final View v) {
                behavior.setState(BottomSheetBehavior.STATE_EXPANDED);
            }
        });

        final View bottomSheet = findViewById(R.id.bottom_sheet);
        behavior = BottomSheetBehavior.from(bottomSheet);

        final ViewPager viewPager = (ViewPager) findViewById(R.id.viewPager);
        viewPager.setAdapter(new MyPagerAdapter());

        final TabLayout tabLayout = (TabLayout) findViewById(R.id.tabs);
        tabLayout.setupWithViewPager(viewPager);


    }

    private
    class MyPagerAdapter
            extends PagerAdapter
    {
        @Override
        public
        int getCount() {
            return 15;
        }

        @Override
        public
        Object instantiateItem(final ViewGroup container, final int position) {
            final RecyclerView recyclerView = new RecyclerView(MainActivity.this);

            recyclerView.setLayoutManager(new LinearLayoutManager(MainActivity.this));
            recyclerView.setAdapter(new ItemAdapter());

            container.addView(recyclerView);
            return recyclerView;
        }

        @Override
        public
        boolean isViewFromObject(final View view, final Object object) {
            return view.equals(object);
        }

        @Override
        public
        void destroyItem(final ViewGroup container, final int position, final Object object) {
            container.removeView((View) object);
        }

        @Override
        public
        CharSequence getPageTitle(final int position) {
            return String.valueOf(position);
        }
    }

    public
    class ItemAdapter
            extends RecyclerView.Adapter<ItemAdapter.ViewHolder>
    {

        @Override
        public
        ViewHolder onCreateViewHolder(final ViewGroup parent, final int viewType) {
            return new ViewHolder(new TextView(MainActivity.this));
        }

        @Override
        public
        void onBindViewHolder(final ViewHolder holder, final int position) {
        }

        @Override
        public
        int getItemCount() {
            return 100;
        }

        public
        class ViewHolder
                extends RecyclerView.ViewHolder
        {
            public TextView textView;

            public
            ViewHolder(final View itemView) {
                super(itemView);
                textView = (TextView) itemView;
            }
        }
    }
}

activity_main.xml

<?xml version="1.0" encoding="utf-8"?>
<android.support.design.widget.CoordinatorLayout android:id = "@+id/coordinatorLayout"
    xmlns:android = "http://schemas.android.com/apk/res/android"
    xmlns:app = "http://schemas.android.com/apk/res-auto"
    xmlns:tools = "http://schemas.android.com/tools"
    android:layout_width = "match_parent"
    android:layout_height = "match_parent"
    android:background = "#a3b1ef"
    android:fitsSystemWindows = "true"
    tools:context = ".ui.MainActivity"
    >

    <Button
        android:id = "@+id/btnView"
        android:layout_width = "match_parent"
        android:layout_height = "wrap_content"
        android:text = "Show view"
        app:layout_behavior = "@string/appbar_scrolling_view_behavior"
        />


    <LinearLayout
        android:id = "@+id/bottom_sheet"
        android:layout_width = "match_parent"
        android:layout_height = "400dp"
        android:background = "#fff"
        android:gravity = "center"
        android:orientation = "vertical"
        app:layout_behavior = "@string/bottom_sheet_behavior"
        >


        <android.support.design.widget.TabLayout
            android:id = "@+id/tabs"
            android:layout_width = "match_parent"
            android:layout_height = "wrap_content"
            app:tabMode = "scrollable"
            />

        <android.support.v4.view.ViewPager
            android:id = "@+id/viewPager"
            android:layout_width = "match_parent"
            android:layout_height = "match_parent"
            />

    </LinearLayout>
</android.support.design.widget.CoordinatorLayout>

Скриншот

Любые идеи для обходного пути?


person Vitaly    schedule 09.06.2016    source источник


Ответы (7)


Я столкнулся с тем же ограничением, но смог его решить.

Причина описанного вами эффекта заключается в том, что BottomSheetBehavior (начиная с версии 24.2.0) поддерживает только один прокручиваемый дочерний элемент, который идентифицируется во время макета следующим образом:

private View findScrollingChild(View view) {
    if (view instanceof NestedScrollingChild) {
        return view;
    }
    if (view instanceof ViewGroup) {
        ViewGroup group = (ViewGroup) view;
        for (int i = 0, count = group.getChildCount(); i < count; i++) {
            View scrollingChild = findScrollingChild(group.getChildAt(i));
            if (scrollingChild != null) {
                return scrollingChild;
            }
        }
    }
    return null;
}

Вы можете видеть, что он по существу находит первого дочернего элемента с прокруткой, используя DFS.

Я немного улучшил эту реализацию и собрал небольшую библиотеку, а также пример приложения. Вы можете найти его здесь: https://github.com/laenger/ViewPagerBottomSheet

Просто добавьте URL-адрес репозитория maven в свой build.gradle:

repositories {
    maven { url "https://raw.github.com/laenger/maven-releases/master/releases" }
}

Добавьте библиотеку в зависимости:

dependencies {
    compile "biz.laenger.android:vpbs:0.0.2"
}

Используйте ViewPagerBottomSheetBehavior для просмотра нижнего листа:

app:layout_behavior="@string/view_pager_bottom_sheet_behavior"

Настройте любой вложенный ViewPager внутри нижнего листа:

BottomSheetUtils.setupViewPager(bottomSheetViewPager)

(Это также работает, когда ViewPager является нижним представлением листа и для дополнительных вложенных ViewPager)

пример реализации

person laenger    schedule 09.07.2016
comment
Отличный ответ. Однако не работает, если BottomSheetDialogFragment используется как BottomSheet. Посмотрите эту тему, где описан такой случай. - person R. Zagórski; 08.09.2016
comment
это действительно интересное дополнение! не могли бы вы открыть запрос на включение в мой репозиторий, чтобы избежать дублирования общего кода? github.com/laenger/ViewPagerBottomSheet - person laenger; 08.09.2016
comment
просто потрясающе. - person eRaisedToX; 07.04.2017
comment
Я тоже сталкиваюсь с этой проблемой в 26.1. Код метода не изменился. Есть ли лучший способ сделать это? Если я включу NestedScrolling для viewPager и для recyclerviews, то прокрутка будет работать на нижнем листе для обоих recyclerviews. Но я не уверен в каких-либо других побочных эффектах. Я думаю, это повлияет на смахивание, чтобы закрыть поведение - person Manas Chaudhari; 21.02.2018
comment
Что ж, на будущее - не могли бы вы немного объяснить, в чем заключалось ваше исправление? У меня есть свое собственное поведение, и я не могу использовать ваше, поэтому я вынужден анализировать и копировать вашу работу самостоятельно. - person Kirill Starostin; 31.07.2018
comment
Да, ключевые изменения видны в Commit 2775715 указанной библиотеки. - person laenger; 31.07.2018
comment
вы решили мою большую проблему. Работает нормально. Большое спасибо @laenger! - person Ajay; 27.12.2019
comment
Это сработало и для нижнего листа. Но как я могу установить состояние нижнего листа в расширенное? так как теперь он показывает ошибку при инициализации объекта BottomSheetBehavoir. @Ленгер - person Srushti Suvarna; 10.03.2020

Этот пост спас мне жизнь: https://medium.com/@hanru.yeh/funny-solution-that-makes-bottomsheetdialog-support-viewpager-with-nestedscrollingchilds-bfdca72235c3

Покажите мое исправление для ViewPager внутри нижнего листа.

package com.google.android.material.bottomsheet

import android.view.View
import androidx.annotation.VisibleForTesting
import androidx.viewpager.widget.ViewPager
import java.lang.ref.WeakReference


class BottomSheetBehaviorFix<V : View> : BottomSheetBehavior<V>(), ViewPager.OnPageChangeListener {

    override fun onPageScrollStateChanged(state: Int) {}

    override fun onPageScrolled(position: Int, positionOffset: Float, positionOffsetPixels: Int) {}

    override fun onPageSelected(position: Int) {
        val container = viewRef?.get() ?: return
        nestedScrollingChildRef = WeakReference(findScrollingChild(container))
    }

    @VisibleForTesting
    override fun findScrollingChild(view: View): View? {
        return if (view is ViewPager) {
            view.focusedChild?.let { findScrollingChild(it) }
        } else {
            super.findScrollingChild(view)
        }
    }
}
person Community    schedule 04.01.2019

Существует еще один подход, который не требует изменения BottomSheetBehavior, а вместо этого использует тот факт, что BottomSheetBehavior распознает только первый найденный NestedScrollView с NestedScrollingEnabled. Поэтому вместо того, чтобы изменять эту логику внутри BottomSheetBehavior, включайте и отключайте соответствующие виды прокрутки. Я обнаружил этот подход здесь: https://imnotyourson.com/cannot-scroll-scrollable-content-inside-viewpager-as-bottomsheet-of-coordinatorlayout/

В моем случае мой BottomSheetBehavior использовал TabLayout с FragmentPagerAdapter, поэтому моему FragmentPagerAdapter требовался следующий код:

@Override
public void setPrimaryItem(@NonNull ViewGroup container, int position, @NonNull Object object) {

        super.setPrimaryItem(container, position, object);

        Fragment f = ((Fragment)object);
        String activeFragmentTag = f.getTag();
        View view = f.getView();

        if (view != null) {
            View nestedView = view.findViewWithTag("nested");               
            if ( nestedView != null && nestedView instanceof NestedScrollView) {
                ((NestedScrollView)nestedView).setNestedScrollingEnabled(true);
            }
        }

        FragmentManager fm = f.getFragmentManager();

        for(Fragment frag : fm.getFragments()) {

            if (frag.getTag() != activeFragmentTag) {
                View v = frag.getView();
                if (v!= null) {

                    View nestedView = v.findViewWithTag("nested");

                    if (nestedView!= null && nestedView instanceof NestedScrollView) {
                        ((NestedScrollView)nestedView).setNestedScrollingEnabled(false);
                    }
                }
            }
        }

        container.requestLayout();
    }

Любые вложенные представления прокрутки в ваших фрагментах просто должны иметь тег «вложенный».

Вот пример файла макета фрагмента:

<?xml version="1.0" encoding="utf-8"?>
<androidx.constraintlayout.widget.ConstraintLayout
    xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    tools:context=".ui.MLeftFragment">

    <androidx.core.widget.NestedScrollView
        android:id="@+id/nestedScrollView"
        android:layout_width="match_parent"
        android:layout_height="match_parent"
        android:tag="nested"
        android:fillViewport="true">

        <LinearLayout
            android:layout_width="match_parent"
            android:layout_height="match_parent"
            android:orientation="vertical">

            <!-- TODO: Update blank fragment layout -->
            <TextView
                android:layout_width="match_parent"
                android:layout_height="wrap_content"
                android:text="@string/hello_mool_left_fragment" />      

        </LinearLayout>  

    </androidx.core.widget.NestedScrollView>

</androidx.constraintlayout.widget.ConstraintLayout>
person voam    schedule 08.01.2019
comment
вау мужик!!! вы сэкономили мне много времени! большое спасибо. это решило мою проблему, ура - person Ivan Karpiuk; 13.02.2019

Я также недавно был в этой ситуации, и я использовал следующий пользовательский класс viewpager вместо viewpager (на XML), и он работал очень хорошо, я думаю, что это поможет вам и другим):


import android.content.Context
import android.util.AttributeSet
import android.view.View
import androidx.viewpager.widget.ViewPager
import java.lang.reflect.Field

class BottomSheetViewPager(context: Context, attrs: AttributeSet?) : ViewPager(context, attrs) {
    constructor(context: Context) : this(context, null)
    private val positionField: Field =
        ViewPager.LayoutParams::class.java.getDeclaredField("position").also {
            it.isAccessible = true
        }

    init {
        addOnPageChangeListener(object : SimpleOnPageChangeListener() {
            override fun onPageSelected(position: Int) {
                requestLayout()
            }
        })
    }

    override fun getChildAt(index: Int): View {
        val stackTrace = Throwable().stackTrace
        val calledFromFindScrollingChild = stackTrace.getOrNull(1)?.let {
            it.className == "com.google.android.material.bottomsheet.BottomSheetBehavior" &&
                    it.methodName == "findScrollingChild"
        }
        if (calledFromFindScrollingChild != true) {
            return super.getChildAt(index)
        }

        val currentView = getCurrentView() ?: return super.getChildAt(index)
        return if (index == 0) {
            currentView
        } else {
            var view = super.getChildAt(index)
            if (view == currentView) {
               view = super.getChildAt(0)
            }
            return view
        }
    }

    private fun getCurrentView(): View? {
        for (i in 0 until childCount) {
            val child = super.getChildAt(i)
            val lp = child.layoutParams as? ViewPager.LayoutParams
            if (lp != null) {
                val position = positionField.getInt(lp)
                if (!lp.isDecor && currentItem == position) {
                    return child
                }
            }
        }
        return null
    }
}

person Muhammad Husniddin Madaminov    schedule 18.08.2019
comment
Элегантное решение, простое в реализации. Спасибо :) - person Qazi Fahim Farhan; 16.01.2020
comment
Вы спасли мой день. Так легко реализовать - person ysfcyln; 31.01.2020
comment
Спасибо, я рад помочь. StackOverflow дал мне так много... - person markomoreno; 08.09.2020

У меня есть решение для AndroidX, Kotlin. Проверено и работает на "com.google.android.material:material:1.1.0-alpha06".

Я также использовал это: СРЕДНИЙ БЛОГ в качестве руководства.

Вот мой класс Kotlin ViewPagerBottomSheetBehavior:

package com.google.android.material.bottomsheet
import android.content.Context
import android.util.AttributeSet
import android.view.View
import androidx.annotation.VisibleForTesting
import androidx.viewpager.widget.ViewPager
import java.lang.ref.WeakReference
class ViewPagerBottomSheetBehavior<V : View>
    : com.google.android.material.bottomsheet.BottomSheetBehavior<V>,
    ViewPager.OnPageChangeListener {

    constructor() : super()
    constructor(context: Context, attrs: AttributeSet) : super(context, attrs)

    override fun onPageScrollStateChanged(state: Int) {}
    override fun onPageScrolled(position: Int, positionOffset: Float, positionOffsetPixels: Int) {}
    override fun onPageSelected(position: Int) {
        val container = viewRef?.get() ?: return
        nestedScrollingChildRef = WeakReference(findScrollingChild(container))
    }

    @VisibleForTesting
    override fun findScrollingChild(view: View?): View? {
        return if (view is ViewPager) {
            view.focusedChild?.let { findScrollingChild(it) }
        } else {
            super.findScrollingChild(view)
        }
    }
}

Последним решением было добавление суперконструкторов в класс:

constructor() : super()
constructor(context: Context, attrs: AttributeSet) : super(context, attrs)

Помните, что вы должны добавить ViewPagerBottomSheetBehavior Kotlin Class по следующему пути: Path Ссылка на изображение класса, потому что вы должны переопределить закрытый метод>

@VisibleForTesting
override fun findScrollingChild(view: View?): View? {
    return if (view is ViewPager) {
        view.focusedChild?.let { findScrollingChild(it) }
    } else {
        super.findScrollingChild(view)
    }
}

После этого вы можете использовать его в качестве атрибута View, вот так >

        <androidx.constraintlayout.widget.ConstraintLayout
          app:layout_behavior="com.google.android.material.bottomsheet.ViewPagerBottomSheetBehavior"
            android:layout_height="match_parent"
            android:layout_width="match_parent">
        <include
                android:layout_width="match_parent"
                android:layout_height="wrap_content"
                layout="@layout/you_content_with_a_viewPager_scroll"
        />
    </androidx.constraintlayout.widget.ConstraintLayout>
person markomoreno    schedule 13.06.2019

Похоже, все, что требуется, — это соответствующим образом обновить nestedScrollingChildRef.

У меня работает простая установка параметра target в onStartNestedScroll:

package com.google.android.material.bottomsheet

class ViewPagerBottomSheetBehavior<V : View>(context: Context, attrs: AttributeSet?) : BottomSheetBehavior<V>(context, attrs) {

    override fun onStartNestedScroll(coordinatorLayout: CoordinatorLayout, child: V, directTargetChild: View, target: View, axes: Int, type: Int): Boolean {
        nestedScrollingChildRef = WeakReference(target)
        return super.onStartNestedScroll(coordinatorLayout, child, directTargetChild, target, axes, type)
    }
}
person Nolan Amy    schedule 04.09.2020
comment
В моем случае это сработало лучше, чем другие решения, из-за сложности и поведения прокрутки макетов на вкладках моего ViewPager. - person Erik Pedersen; 17.12.2020

Предполагая, что page является NestedScrollView, я смог решить проблему, переключая его свойство isNestedScrollingEnabled в зависимости от того, является ли это входящей или исходящей страницей.

val viewPager = findViewById<ViewPager>(R.id.viewPager)

viewPager.setPageTransformer(false) { page, position ->
    if (position == 0.0f) {
        page.isNestedScrollingEnabled = true
    } else if (position % 1 == 0.0f) {
        page.isNestedScrollingEnabled = false
    }
}
person Nate Whittaker    schedule 17.06.2020