yanzhenjie/SwipeRecyclerView

BaseRecyclerViewAdapterHelper 点击事件兼容问题

Open

#85 aberto em 11 de ago. de 2017

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)Java (1.095 forks)batch import
enhancementhelp wanted

Métricas do repositório

Stars
 (5.544 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

不好意思严大，虽然按道理不建议两个库混用，但是还是有一些小问题，麻烦帮忙看一下，不一定需要解决。 image

在adapte中未添加item的父布局情况下（如下） image mRecyclerView.setSwipeItemClickListener方法 会拦截animationAdapter.setOnItemClickListener方法。

即打印：
KLog.e("=======0==========" + position);

当在 adapter 中使用了 helper.addOnClickListener(R.id.card_view); ，mRecyclerView.setSwipeItemClickListener方法 和 animationAdapter.setOnItemClickListener方法， 都失效，只打印：
KLog.e("=======3====onItemChildClick===========");

在不使用 mRecyclerView.setSwipeItemClickListener方法 的时候，mRecyclerView.setSwipeItemClickListener方法 和 animationAdapter.setOnItemClickListener方法， 都失效 image

Guia do colaborador

Direção de pesquisa
Investigue o tratamento de eventos de clique em BaseRecyclerViewAdapterHelper e SwipeRecyclerView. Determine por que setSwipeItemClickListener intercepta setOnItemClickListener e como addOnClickListener afeta o comportamento. Compare o código fonte de ambas as bibliotecas para entender a ordem de registro dos listeners.
Pilha de tecnologia
javaandroid
Domain
mobile
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Aguardando mantenedor
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Android developmentJavaRecyclerView
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40