From c3117d08bf8d074e61e4bc1049212c0a528b4931 Mon Sep 17 00:00:00 2001 From: Sam Bosley Date: Thu, 7 Feb 2013 17:57:40 -0800 Subject: [PATCH] Fixed task edit page checkbox appearance bug --- .../com/todoroo/astrid/ui/EditTitleControlSet.java | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/astrid/src/com/todoroo/astrid/ui/EditTitleControlSet.java b/astrid/src/com/todoroo/astrid/ui/EditTitleControlSet.java index 22760a79a..618b4b90e 100644 --- a/astrid/src/com/todoroo/astrid/ui/EditTitleControlSet.java +++ b/astrid/src/com/todoroo/astrid/ui/EditTitleControlSet.java @@ -64,6 +64,7 @@ public class EditTitleControlSet extends TaskEditControlSet implements Importanc scaleAnimation.setDuration(100); // set check box to actual action item state completeBox.startAnimation(scaleAnimation); + updateCompleteBox(); } }); } @@ -101,14 +102,17 @@ public class EditTitleControlSet extends TaskEditControlSet implements Importanc private void updateCompleteBox() { + boolean checked = completeBox.isChecked(); + int[] resourceArray = isRepeating ? (checked ? TaskAdapter.IMPORTANCE_REPEAT_RESOURCES_CHECKED : TaskAdapter.IMPORTANCE_REPEAT_RESOURCES) + : (checked ? TaskAdapter.IMPORTANCE_RESOURCES_CHECKED : TaskAdapter.IMPORTANCE_RESOURCES); int valueToUse = importanceValue; - if (valueToUse >= TaskAdapter.IMPORTANCE_RESOURCES.length) - valueToUse = TaskAdapter.IMPORTANCE_RESOURCES.length - 1; - if(valueToUse < TaskAdapter.IMPORTANCE_RESOURCES.length) { + if (valueToUse >= resourceArray.length) + valueToUse = resourceArray.length - 1; + if(valueToUse < resourceArray.length) { if (isRepeating) { - completeBox.setImageResource(TaskAdapter.IMPORTANCE_REPEAT_RESOURCES[valueToUse]); + completeBox.setImageResource(resourceArray[valueToUse]); } else { - completeBox.setImageResource(TaskAdapter.IMPORTANCE_RESOURCES[valueToUse]); + completeBox.setImageResource(resourceArray[valueToUse]); } } }