Closed GoogleCodeExporter closed 9 years ago
I just encountered similar problem. it seems like it's trying to modify root
event instead of very first action, and it throws error accordingly.
Original comment by sv767%ny...@gtempaccount.com
on 24 Jul 2010 at 3:10
I think this is duplicate of issue69
Original comment by sv767%ny...@gtempaccount.com
on 24 Jul 2010 at 3:40
I think this is slightly different from Issue 69. In this bug, you just need to
click on somewhere else other than your intended target and use the long click
to edit. It seems that the focus remains unchanged after doing long click so
the activity becomes confused. However, the context menu was not yet existent
when issue 69 was reported...
Original comment by renc...@gmail.com
on 24 Jul 2010 at 3:48
Agreed, this is different. My guess is that when you longclick it's performing
the context menu on "getItemSelected" which is actually the last item you
clicked. In this case, the Filter. For some reason it passes the
getItemLongClicked to the next dialog though. Shouldn't be that hard to fix.
Original comment by case.and...@gmail.com
on 24 Jul 2010 at 4:54
Original comment by renc...@gmail.com
on 25 Jul 2010 at 11:01
Code review:
http://codereview.appspot.com/1871053
Caused by:
The focus does not change whenever a long-click is performed, so when you
finish the action and return to the activity, the onActivityResult will still
think that the last item clicked is the one you performed that action on with
long click.
Fix detail:
Whenever a long click is performed on an item, that item is also set to be
"selected".
Test Cases:
1. Create a new rule with multiple actions/filters.
1.1 Click on an action and perform a long-click on another one. Select "delete" on the context menu. The correct item should be deleted.
1.2 Click on an action and perform a long-click on another one. Select "edit" on the context menu. After finishing the edit, the changes should reflect on the proper action.
Original comment by renc...@gmail.com
on 28 Jul 2010 at 9:55
Checked-in @ r836.
Original comment by renc...@gmail.com
on 28 Jul 2010 at 11:10
Original comment by renc...@gmail.com
on 29 Jul 2010 at 8:37
Original issue reported on code.google.com by
renc...@gmail.com
on 22 Jul 2010 at 4:02