From f7143cc2ffa2e3628d11d1c1c088d0746430106c Mon Sep 17 00:00:00 2001 From: Sam Bosley Date: Thu, 23 Feb 2012 16:16:50 -0800 Subject: [PATCH] Fixed an issue where swiping list area below task could trigger indent and drop events --- .../com/todoroo/astrid/subtasks/OrderedListFragmentHelper.java | 2 ++ 1 file changed, 2 insertions(+) diff --git a/astrid/plugin-src/com/todoroo/astrid/subtasks/OrderedListFragmentHelper.java b/astrid/plugin-src/com/todoroo/astrid/subtasks/OrderedListFragmentHelper.java index 2fcfd0b44..cba78b6bc 100644 --- a/astrid/plugin-src/com/todoroo/astrid/subtasks/OrderedListFragmentHelper.java +++ b/astrid/plugin-src/com/todoroo/astrid/subtasks/OrderedListFragmentHelper.java @@ -129,6 +129,7 @@ public class OrderedListFragmentHelper { @Override public void drop(int from, int to) { long targetTaskId = taskAdapter.getItemId(from); + if (targetTaskId <= 0) return; // This can happen with gestures on empty parts of the list (e.g. extra space below tasks) long destinationTaskId = taskAdapter.getItemId(to); try { @@ -158,6 +159,7 @@ public class OrderedListFragmentHelper { protected void indent(int which, int delta) { long targetTaskId = taskAdapter.getItemId(which); + if (targetTaskId <= 0) return; // This can happen with gestures on empty parts of the list (e.g. extra space below tasks) try { updater.indent(getFilter(), list, targetTaskId, delta); } catch (Exception e) {