ViewPager dentro de um ScrollView não rola corretamente

88

Eu tenho uma 'página' que tem vários componentes, e seu conteúdo é maior do que a altura do dispositivo. Tudo bem, basta colocar todo o layout (a página inteira) dentro de um ScrollView, sem problemas.

Um dos componentes é a ViewPager. Isso é renderizado corretamente, mas a resposta a um golpe / arremesso não está funcionando corretamente, é instável e nem sempre funciona. Parece estar ficando 'confuso' com o ScrollView, então só funciona 100% quando você arremessa em uma linha horizontal exata.

Se eu remover o ScrollView, o ViewPager responde perfeitamente.

Eu fiz uma pesquisa e não encontrei isso como um defeito conhecido. Alguém mais experimentou isso?

  • Versão da plataforma: 1.6
  • Biblioteca de compatibilidade v4.
  • Dispositivo: HTC Incredible S

Abaixo está algum código de exemplo para você testar, comente ScrollViewpara ver se está funcionando corretamente.

Atividade:

package com.ss.activities;

import com.ss.R;

import android.app.Activity;
import android.content.Context;
import android.graphics.Color;
import android.os.Bundle;
import android.os.Parcelable;
import android.support.v4.view.PagerAdapter;
import android.support.v4.view.ViewPager;
import android.view.View;
import android.widget.TextView;

public class PagerInsideScollViewActivity extends Activity {
    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.main);

        ViewPager vp = (ViewPager) findViewById(R.id.viewpager);
        vp.setAdapter(new MyPagerAdapter(this));
    }
}

class MyPagerAdapter extends PagerAdapter {

    private Context ctx;

    public MyPagerAdapter(Context context) {
        ctx = context;
    }

    @Override
    public int getCount() {
        return 2;
    }

    @Override
    public Object instantiateItem(View collection, int position) {

        TextView tv =  new TextView(ctx);
        tv.setTextSize(50);
        tv.setTextColor(Color.WHITE);
        tv.setText("SMILE DUDE, SMILE DUDE, SMILE DUDE, SMILE DUDE, SMILE DUDE, " +
                "SMILE DUDE, SMILE DUDE, SMILE DUDE, SMILE DUDE, SMILE DUDE, " +
                "SMILE DUDE, SMILE DUDE, SMILE DUDE, SMILE DUDE, SMILE DUDE, " +
                "SMILE DUDE, SMILE DUDE, SMILE DUDE");

        ((ViewPager) collection).addView(tv);

        return tv;

    }

    @Override
    public void destroyItem(View collection, int position, Object view) {
         ((ViewPager) collection).removeView((View) view);
    }

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

    @Override
    public Parcelable saveState() {
        return null;
    }

    @Override
    public void restoreState(Parcelable arg0, ClassLoader arg1) {
    }

    @Override
    public void startUpdate(View arg0) {
    }

    @Override
    public void finishUpdate(View arg0) {
    }
}

Layout:

<?xml version="1.0" encoding="utf-8"?>
    <ScrollView
         xmlns:android="http://schemas.android.com/apk/res/android"
        android:layout_width="fill_parent"
        android:layout_height="fill_parent" >

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

            <android.support.v4.view.ViewPager
                android:id="@+id/viewpager"
                android:layout_width="fill_parent"
                android:layout_height="300dp" />

        </LinearLayout>

    </ScrollView>
electricSunny
fonte
Verifique a resposta aqui: stackoverflow.com/questions/9034030/viewpager-in-scrollview/…
Andrew Dmytrenko
Por favor, mostre minha resposta neste link: stackoverflow.com/questions/7381360/…
Geral Alexander Torres

Respostas:

60

Eu tive o mesmo problema. Minha solução foi chamar requestDisallowInterceptTouchEventquando a rolagem do ViewPager começou.

Aqui está o meu código:

pager.setOnTouchListener(new View.OnTouchListener() {
    @Override
    public boolean onTouch(View v, MotionEvent event) {
        v.getParent().requestDisallowInterceptTouchEvent(true);
        return false;
    }
});

pager.setOnPageChangeListener(new SimpleOnPageChangeListener() {
    @Override
    public void onPageScrolled(int position, float positionOffset, int positionOffsetPixels) {
        pager.getParent().requestDisallowInterceptTouchEvent(true);
    }
});
O Melhor Artista
fonte
1
isso está funcionando. Eu gostaria que a visualização de rolagem fosse verticalmente, mesmo que o toque estivesse iniciando no pager de visualização, se o movimento fosse "principalmente" vertical
Nemanja Kovacevic
3
é necessário ter requestDisallowInterceptTouchEvent em onTouch e onPageScrolled ou isso é apenas um exagero? Poderíamos obter os mesmos resultados movendo requestDisallowInterceptTouchEvent para apenas onPageScrollStateChanged?
craigrs84
1
Estou com o mesmo problema, e encontrei esta melhor solução, espero que funcione para você. bitbucket.org/NxAlex/swipes-navigation-demo/src/…
Jitendra Nath
como posso ainda substituir o evento onClick? está em conflito com no Touch?
Kiddo
4
qual é o pager e o mpager aqui que preciso dar a entrada?
Sujithrao
30

Leituras adicionais revelaram que há problemas com os componentes de rolagem dentro dos componentes de rolagem.

Uma solução é 'desativar' a rolagem vertical do ScrollView na área do componente rolável contido, no meu caso um ViewPager.

O código para esta solução é detalhado aqui (e é simplesmente brilhante!)

electricSunny
fonte
qual solução nessa página devo referir?
Harsha MV
1
requestDisallowInterceptTouchEvent é o que você procura.
yahya
1
Estou com o mesmo problema, e encontrei esta melhor solução, espero que funcione para você. bitbucket.org/NxAlex/swipes-navigation-demo/src/…
Jitendra Nath
obrigado pelo link, a solução que encontrei lá ainda me deu problemas com deslizamentos horizontais não perfeitamente no viewpager, então eu modifiquei um pouco para fazê-lo funcionar: stackoverflow.com/a/33696740/2093236
Dmide
15

Aqui está uma solução:

    mPager.setOnTouchListener(new View.OnTouchListener() {

        int dragthreshold = 30;
        int downX;
        int downY;

        @Override
        public boolean onTouch(View v, MotionEvent event) {

            switch (event.getAction()) {
            case MotionEvent.ACTION_DOWN:
                downX = (int) event.getRawX();
                downY = (int) event.getRawY();
                break;
            case MotionEvent.ACTION_MOVE:
                int distanceX = Math.abs((int) event.getRawX() - downX);
                int distanceY = Math.abs((int) event.getRawY() - downY);

                if (distanceY > distanceX && distanceY > dragthreshold) {
                    mPager.getParent().requestDisallowInterceptTouchEvent(false);
                    mScrollView.getParent().requestDisallowInterceptTouchEvent(true);
                } else if (distanceX > distanceY && distanceX > dragthreshold) {
                    mPager.getParent().requestDisallowInterceptTouchEvent(true);
                    mScrollView.getParent().requestDisallowInterceptTouchEvent(false);
                }
                break;
            case MotionEvent.ACTION_UP:
                mScrollView.getParent().requestDisallowInterceptTouchEvent(false);
                mPager.getParent().requestDisallowInterceptTouchEvent(false);
                break;
            }
            return false;
        }
    });

Basicamente definindo os valores X, Y ao pressionar para baixo e calculando a distância enquanto arrasta para determinar o caminho que gostaríamos de seguir. Brinque com o limite de arrasto para otimizar o seu caso.


fonte
Eu tive o mesmo problema. Esta é a única solução que funcionou. Bom trabalho ... :)
rawcoder064
Esta é a melhor solução para mim. Isso não perturbará o ScrollView deslizando para cima e para baixo.
Lei Guo
Esta é a melhor solução para mim. Também temos a opção de configurar o valor do limite nesta solução.
Vinayak
não funciona para mim, porque o scrollview intercepta o toque antes que esta declaração de cancelamento seja alcançada
AdrianoCelentano
Excelente trabalho, e se você quiser usar isso com Wrapcontentviewpager, então também está funcionando muito bem
Silvans Solanki
4

Com um ViewPager, você pode capturar eventos de mudança de página e evitar que o ScrollView intercepte o evento de toque que causou a mudança de página.

Isso é muito simples, usando ViewGroup.requestDisallowInterceptTouchEvent(boolean). Ele também permite que o usuário arraste o ScrollView mesmo se começar a arrastar no ViewPager, mas o arrasto horizontal no pager ainda funcionará sem a interferência do ScrollView.

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.main);

        // Add android:id for your ScrollView in your layout
        final ScrollView sv = (ScrollView) findViewById(R.id.scrollview);
        final ViewPager vp = (ViewPager) findViewById(R.id.viewpager);
        vp.setAdapter(new MyPagerAdapter(this));

        // Use a page-change listener to respond to begin-drag events:
        vp.setOnPageChangeListener(new OnPageChangeListener() {
            @Override
            public void onPageSelected(int newState) {
                if (newState == ViewPager.SCROLL_STATE_DRAGGING) {
                    // Prevent the ScrollView from intercepting this event now that the page is changing.
                    // When this drag ends, the ScrollView will start accepting touch events again.
                    sv.requestDisallowInterceptTouchEvent(true);
                }
            }

            @Override
            public void onPageScrolled(int arg0, float arg1, int arg2) {
            }

            @Override
            public void onPageScrollStateChanged(int arg0) {
            }
        });

    }

Isso funciona para mim com a biblioteca Android Support v4 no Android 2.3.4 e 4.2.1.

mrb
fonte
1
deveria ser onPageScrollStateChanged em vez de onPageSelected?
craigrs84
2

Eu adaptei a solução de @Michael Herbig. A vantagem disso é que ela funciona em qualquer visualização que permitir setOnTouchListener, não apenas em um ViewPager (por exemplo ViewPagerIndicator) e é uma classe independente.

Exemplo de uso:

// runStatsPager is a android.support.v4.view.ViewPager;
runStatsPager.setOnTouchListener(new ViewInScrollViewTouchHelper(runStatsPager));

// runStatsPagerIndicator is a com.viewpagerindicator.TitlePageIndicator
runStatsPagerIndicator.setOnTouchListener(new ViewInScrollViewTouchHelper(runStatsPagerIndicator));

E a classe:

class ViewInScrollViewTouchHelper implements View.OnTouchListener {

    private final ScrollView scrollView;
    private final View viewInScrollView;
    int dragthreshold = 30;
    int downX;
    int downY;

    public ViewInScrollViewTouchHelper(View viewInScrollView) {

        if (viewInScrollView == null) {
            throw new IllegalArgumentException("viewInScrollView cannot be null.");
        }

        ViewParent parent = viewInScrollView.getParent();
        ScrollView scrollView = null;
        do {
            if (parent instanceof ScrollView) {
                scrollView = (ScrollView) parent;
                break;
            }
        } while(parent != null && (parent = parent.getParent()) != null);

        if (scrollView == null) {
            throw new IllegalArgumentException("View does not have a ScrollView in its parent hierarchy.");
        }

        this.scrollView = scrollView;
        this.viewInScrollView = viewInScrollView;
    }

    @Override
    public boolean onTouch(View v, MotionEvent event) {

        switch (event.getAction()) {
            case MotionEvent.ACTION_DOWN:
                downX = (int) event.getRawX();
                downY = (int) event.getRawY();
                break;
            case MotionEvent.ACTION_MOVE:
                int distanceX = Math.abs((int) event.getRawX() - downX);
                int distanceY = Math.abs((int) event.getRawY() - downY);

                if (distanceY > distanceX && distanceY > dragthreshold) {
                    viewInScrollView.getParent().requestDisallowInterceptTouchEvent(false);
                    scrollView.getParent().requestDisallowInterceptTouchEvent(true);
                } else if (distanceX > distanceY && distanceX > dragthreshold) {
                    viewInScrollView.getParent().requestDisallowInterceptTouchEvent(true);
                    scrollView.getParent().requestDisallowInterceptTouchEvent(false);
                }
                break;
            case MotionEvent.ACTION_UP:
                scrollView.getParent().requestDisallowInterceptTouchEvent(false);
                viewInScrollView.getParent().requestDisallowInterceptTouchEvent(false);
                break;
        }
        return false;
    }
}
Jon Willis
fonte
1
public class WrapContentHeightViewPager extends ViewPager {

public WrapContentHeightViewPager(Context context) {
    super(context);
}

public WrapContentHeightViewPager(Context context, AttributeSet attrs) {
    super(context, attrs);
}

@Override
protected void onMeasure(int widthMeasureSpec, int heightMeasureSpec) {
    super.onMeasure(widthMeasureSpec, heightMeasureSpec);

    int height = 0;
    for (int i = 0; i < getChildCount(); i++) {
        View child = getChildAt(i);
        child.measure(widthMeasureSpec, View.MeasureSpec.makeMeasureSpec(0, MeasureSpec.UNSPECIFIED));

        int h = child.getMeasuredHeight();
        if (h > height) height = h;
    }

    heightMeasureSpec = MeasureSpec.makeMeasureSpec(height, MeasureSpec.EXACTLY);

    super.onMeasure(widthMeasureSpec, heightMeasureSpec);
}

}

@Override public boolean onTouch (View v, MotionEvent event) {

    int dragthreshold = 30;
    int downX = 0;
    int downY = 0;

    switch (event.getAction()) {
        case MotionEvent.ACTION_DOWN:
            downX = (int) event.getRawX();
            downY = (int) event.getRawY();
            break;
        case MotionEvent.ACTION_MOVE:
            int distanceX = Math.abs((int) event.getRawX() - downX);
            int distanceY = Math.abs((int) event.getRawY() - downY);

            if (distanceY > distanceX && distanceY > dragthreshold) {
                mViewPager.getParent().requestDisallowInterceptTouchEvent(false);
                mScrollView.getParent().requestDisallowInterceptTouchEvent(true);
            } else if (distanceX > distanceY && distanceX > dragthreshold) {
                mViewPager.getParent().requestDisallowInterceptTouchEvent(true);
                mScrollView.getParent().requestDisallowInterceptTouchEvent(false);
            }
            break;
        case MotionEvent.ACTION_UP:
            mScrollView.getParent().requestDisallowInterceptTouchEvent(false);
            mViewPager.getParent().requestDisallowInterceptTouchEvent(false);
            break;
    }
    return false;

}

Use os dois acima e funcionará muito bem. Eu usei em meu código também.

Silvans Solanki
fonte
1

Portanto, com essa abordagem, deixo o ViewPagerpergaminho na Xdireção sem ter que me preocupar com o ScrollView(pai) roubando o evento e cancelando o pergaminho atual. Mais importante, isso também deixa a área onde o ViewPagerestá localizado rolável na Ydireção.

public class CustomViewPager extends ViewPager {

private GestureDetector     mGestureDetector;
View.OnTouchListener        mGestureListener;

public CustomViewPager(Context context, AttributeSet attrs) {
    super(context, attrs);
    mGestureDetector = new GestureDetector(context, new Detector());
}

@Override
public boolean onTouchEvent(MotionEvent motionEvent) {

    mGestureDetector.onTouchEvent(motionEvent);
    return super.onTouchEvent(motionEvent);
}

class Detector extends SimpleOnGestureListener {

    @Override
    public boolean onScroll(MotionEvent e1, MotionEvent e2, float distanceX, float distanceY) {

        requestDisallowInterceptTouchEvent(true);

        return false;
    }
}
}
Slickelito
fonte
É uma pena criar uma subclasse de ViewPager aqui. Eu imagino que isso poderia ser adaptado para um View.OnTouchListener. Vou votar a favor por agora!
Jon Willis
0

Nenhuma das soluções aqui funcionou bem para mim porque é a modificação da ViewPagerlógica de toque de ou ScrollViewda lógica de. Tive que implementar os dois, agora funciona perfeitamente.

public class TouchGreedyViewPager extends ViewPager {
    private float xDistance, yDistance, lastX, lastY;

    public TouchGreedyViewPager(Context context, AttributeSet attrs) {
        super(context, attrs);
    }

    @Override
    public boolean onInterceptTouchEvent(MotionEvent ev) {
        switch (ev.getAction()) {
            case MotionEvent.ACTION_DOWN:
                xDistance = yDistance = 0f;
                lastX = ev.getX();
                lastY = ev.getY();
                break;
            case MotionEvent.ACTION_MOVE:
                final float curX = ev.getX();
                final float curY = ev.getY();
                xDistance += Math.abs(curX - lastX);
                yDistance += Math.abs(curY - lastY) / 3; // favor X events
                lastX = curX;
                lastY = curY;
                if (xDistance > yDistance) {
                    return true;
                }
        }

        return super.onInterceptTouchEvent(ev);
    }
}

public class TouchHumbleScrollView extends ScrollView {
    private float xDistance, yDistance, lastX, lastY;

    public TouchHumbleScrollView(Context context, AttributeSet attrs) {
        super(context, attrs);
    }

    @Override
    public boolean onInterceptTouchEvent(MotionEvent ev) {
        switch (ev.getAction()) {
            case MotionEvent.ACTION_DOWN:
                xDistance = yDistance = 0f;
                lastX = ev.getX();
                lastY = ev.getY();
                break;
            case MotionEvent.ACTION_MOVE:
                final float curX = ev.getX();
                final float curY = ev.getY();
                xDistance += Math.abs(curX - lastX);
                yDistance += Math.abs(curY - lastY) / 3; // favor X events
                lastX = curX;
                lastY = curY;
                if (xDistance > yDistance) {
                    return false;
                }
        }

        return super.onInterceptTouchEvent(ev);
    }
}
Dmide
fonte
-1

Você pode substituir o método instantiateItem em sua classe PagerAdapter adicionando um scrollView a cada página. Aqui está um código:

@Override
public Object instantiateItem(View collection, int position) {
    ScrollView sc = new ScrollView(cxt);
    sc.setLayoutParams(new LayoutParams(LayoutParams.WRAP_CONTENT, LayoutParams.WRAP_CONTENT));
    sc.setFillViewport(true);
    TextView tv = new TextView(cxt);
    tv.setLayoutParams(new LayoutParams(LayoutParams.WRAP_CONTENT, LayoutParams.WRAP_CONTENT));
    tv.setText(pages[position]);
    tv.setPadding(5,5,5,5);         
    sc.addView(tv);
    ((ViewPager) collection).addView(sc);

    return sc;
    }
Marcin S.
fonte
Resuma ou cite os pontos mais importantes aqui. A podridão do link acontece.
ЯegDwight