Subtasks list fragments should retain position after resuming

astrid/plugin-src/com/todoroo/astrid/subtasks/SubtasksListFragment.java

@@ -26,6 +26,8 @@ public class SubtasksListFragment extends TaskListFragment {
 
     protected OrderedListFragmentHelperInterface<?> helper;
 
+    private int lastVisibleIndex = -1;
+
     @Override
     public void onActivityCreated(Bundle savedInstanceState) {
         helper = createFragmentHelper();
@@ -62,6 +64,20 @@ public class SubtasksListFragment extends TaskListFragment {
         unregisterForContextMenu(getListView());
     }
 
+    @Override
+    public void onPause() {
+        super.onPause();
+        lastVisibleIndex = getListView().getFirstVisiblePosition();
+    }
+
+    @Override
+    public void onResume() {
+        super.onResume();
+        if (lastVisibleIndex >=0) {
+            getListView().setSelection(lastVisibleIndex);
+        }
+    }
+
     @Override
     protected boolean isDraggable() {
         return true;

astrid/plugin-src/com/todoroo/astrid/subtasks/SubtasksTagListFragment.java

@@ -19,6 +19,8 @@ public class SubtasksTagListFragment extends TagViewFragment {
 
     private final AstridOrderedListFragmentHelper<TaskListMetadata> helper;
 
+    private int lastVisibleIndex = -1;
+
     public SubtasksTagListFragment() {
         super();
         helper = new AstridOrderedListFragmentHelper<TaskListMetadata>(this, new SubtasksTagUpdater(isBeingFiltered));
@@ -58,6 +60,20 @@ public class SubtasksTagListFragment extends TagViewFragment {
         unregisterForContextMenu(getListView());
     }
 
+    @Override
+    public void onPause() {
+        super.onPause();
+        lastVisibleIndex = getListView().getFirstVisiblePosition();
+    }
+
+    @Override
+    public void onResume() {
+        super.onResume();
+        if (lastVisibleIndex >=0) {
+            getListView().setSelection(lastVisibleIndex);
+        }
+    }
+
     @Override
     protected boolean isDraggable() {
         return true;