Prevent long press from leaking into header gesture detector

master
M66B 3 weeks ago
parent 8ef55ae3da
commit 935114fc4c

@ -389,6 +389,8 @@ public class FragmentMessages extends FragmentBase
private boolean scrolling = false; private boolean scrolling = false;
private boolean navigating = false; private boolean navigating = false;
private GestureDetector gestureDetector;
private AdapterMessage adapter; private AdapterMessage adapter;
private AdapterMessage.ViewType viewType; private AdapterMessage.ViewType viewType;
@ -1196,67 +1198,65 @@ public class FragmentMessages extends FragmentBase
}; };
rvMessage.addItemDecoration(dateDecorator); rvMessage.addItemDecoration(dateDecorator);
rvMessage.addOnItemTouchListener(new RecyclerView.OnItemTouchListener() { gestureDetector =
private final GestureDetector gestureDetector = new GestureDetector(getContext(), new GestureDetector.SimpleOnGestureListener() {
new GestureDetector(getContext(), new GestureDetector.SimpleOnGestureListener() { @Override
@Override public void onLongPress(@NonNull MotionEvent e) {
public void onLongPress(@NonNull MotionEvent e) { if (!getLifecycle().getCurrentState().isAtLeast(Lifecycle.State.STARTED))
if (!getLifecycle().getCurrentState().isAtLeast(Lifecycle.State.STARTED)) return;
return;
if (swiping)
return;
int x = Math.round(e.getX());
int y = Math.round(e.getY());
Rect rect = new Rect();
for (int i = 0; i < rvMessage.getChildCount(); i++) {
View child = rvMessage.getChildAt(i);
if (child == null)
continue;
dateDecorator.getItemOffsets(rect, child, rvMessage, null); int x = Math.round(e.getX());
if (rect.height() == 0) int y = Math.round(e.getY());
continue;
rect.set(child.getLeft(), child.getTop() - rect.top, child.getRight(), child.getTop()); Rect rect = new Rect();
if (!rect.contains(x, y)) for (int i = 0; i < rvMessage.getChildCount(); i++) {
continue; View child = rvMessage.getChildAt(i);
if (child == null)
continue;
int pos = rvMessage.getChildAdapterPosition(child); dateDecorator.getItemOffsets(rect, child, rvMessage, null);
if (pos == NO_POSITION) if (rect.height() == 0)
continue; continue;
TupleMessageEx message = adapter.getItemAtPosition(pos); rect.set(child.getLeft(), child.getTop() - rect.top, child.getRight(), child.getTop());
if (message == null) if (!rect.contains(x, y))
continue; continue;
Calendar cal = Calendar.getInstance(); int pos = rvMessage.getChildAdapterPosition(child);
cal.setTimeInMillis(message.received); if (pos == NO_POSITION)
cal.set(Calendar.HOUR_OF_DAY, 0); continue;
cal.set(Calendar.MINUTE, 0);
cal.set(Calendar.SECOND, 0);
cal.set(Calendar.MILLISECOND, 0);
if (date_week) {
cal.setMinimalDaysInFirstWeek(4); // ISO 8601
cal.setFirstDayOfWeek(Calendar.MONDAY);
cal.set(Calendar.DAY_OF_WEEK, Calendar.MONDAY);
cal.add(Calendar.DATE, 7);
} else
cal.add(Calendar.DATE, 1);
long to = cal.getTimeInMillis();
cal.add(Calendar.DATE, date_week ? -7 : -1); TupleMessageEx message = adapter.getItemAtPosition(pos);
long from = cal.getTimeInMillis(); if (message == null)
continue;
onMenuSelect(from, to, true); Calendar cal = Calendar.getInstance();
Helper.performHapticFeedback(view, HapticFeedbackConstants.CONFIRM); cal.setTimeInMillis(message.received);
return; cal.set(Calendar.HOUR_OF_DAY, 0);
} cal.set(Calendar.MINUTE, 0);
cal.set(Calendar.SECOND, 0);
cal.set(Calendar.MILLISECOND, 0);
if (date_week) {
cal.setMinimalDaysInFirstWeek(4); // ISO 8601
cal.setFirstDayOfWeek(Calendar.MONDAY);
cal.set(Calendar.DAY_OF_WEEK, Calendar.MONDAY);
cal.add(Calendar.DATE, 7);
} else
cal.add(Calendar.DATE, 1);
long to = cal.getTimeInMillis();
cal.add(Calendar.DATE, date_week ? -7 : -1);
long from = cal.getTimeInMillis();
onMenuSelect(from, to, true);
Helper.performHapticFeedback(view, HapticFeedbackConstants.CONFIRM);
return;
} }
}); }
});
rvMessage.addOnItemTouchListener(new RecyclerView.OnItemTouchListener() {
@Override @Override
public boolean onInterceptTouchEvent(@NonNull RecyclerView rv, @NonNull MotionEvent e) { public boolean onInterceptTouchEvent(@NonNull RecyclerView rv, @NonNull MotionEvent e) {
gestureDetector.onTouchEvent(e); gestureDetector.onTouchEvent(e);
@ -3282,6 +3282,7 @@ public class FragmentMessages extends FragmentBase
@Override @Override
public void run() { public void run() {
swiping = false; swiping = false;
gestureDetector.setIsLongpressEnabled(true);
Log.i("Swiping ended"); Log.i("Swiping ended");
} }
}; };
@ -3292,6 +3293,7 @@ public class FragmentMessages extends FragmentBase
getMainHandler().removeCallbacks(disableSwiping); getMainHandler().removeCallbacks(disableSwiping);
if (actionState == ItemTouchHelper.ACTION_STATE_SWIPE) { if (actionState == ItemTouchHelper.ACTION_STATE_SWIPE) {
swiping = true; swiping = true;
gestureDetector.setIsLongpressEnabled(false);
Log.i("Swiping started"); Log.i("Swiping started");
} else } else
getMainHandler().postDelayed(disableSwiping, ViewConfiguration.getLongPressTimeout() + 100); getMainHandler().postDelayed(disableSwiping, ViewConfiguration.getLongPressTimeout() + 100);

Loading…
Cancel
Save