diff options
author | Adam Cohen <adamcohen@google.com> | 2011-05-18 15:26:57 -0700 |
---|---|---|
committer | Adam Cohen <adamcohen@google.com> | 2011-05-18 15:26:57 -0700 |
commit | 7c6932140dcbd2db88d7a809ba72ee04abb5bf26 (patch) | |
tree | c3a63efb70998c4caa0d23b4d87545d41adcfeda /src/com/android/launcher2/Folder.java | |
parent | f4b08913677e18a8412930972237b91d5a946d95 (diff) | |
download | android_packages_apps_Trebuchet-7c6932140dcbd2db88d7a809ba72ee04abb5bf26.tar.gz android_packages_apps_Trebuchet-7c6932140dcbd2db88d7a809ba72ee04abb5bf26.tar.bz2 android_packages_apps_Trebuchet-7c6932140dcbd2db88d7a809ba72ee04abb5bf26.zip |
Fixing folder bugs:
-> Folders were not loading with ordering properly persisted
-> When an item was put in the trash directly from a folder
it was trying to remove the item from the folder again, sometimes
removing an extra item from the folder
-> Making sure the FolderIcon always shows the _last_ 4 items;
this was not working when the launcher was being restarted
-> Fixed a tiny rotation bug in the FolderIcon
Change-Id: I27423f17fd2f2b396f844c055f1e7abb4f4d5d19
Diffstat (limited to 'src/com/android/launcher2/Folder.java')
-rw-r--r-- | src/com/android/launcher2/Folder.java | 77 |
1 files changed, 44 insertions, 33 deletions
diff --git a/src/com/android/launcher2/Folder.java b/src/com/android/launcher2/Folder.java index a4aeec407..3411f2723 100644 --- a/src/com/android/launcher2/Folder.java +++ b/src/com/android/launcher2/Folder.java @@ -25,7 +25,6 @@ import android.animation.PropertyValuesHolder; import android.animation.ValueAnimator; import android.animation.ValueAnimator.AnimatorUpdateListener; import android.content.Context; -import android.graphics.Matrix; import android.graphics.Rect; import android.util.AttributeSet; import android.view.LayoutInflater; @@ -81,6 +80,8 @@ public class Folder extends LinearLayout implements DragSource, OnItemLongClickL private int mMaxCountX; private int mMaxCountY; private Rect mNewSize = new Rect(); + private ArrayList<View> mItemsInReadingOrder = new ArrayList<View>(); + boolean mItemsInvalidated = false; /** * Used to inflate the Workspace from XML. @@ -213,9 +214,10 @@ public class Folder extends LinearLayout implements DragSource, OnItemLongClickL void bind(FolderInfo info) { mInfo = info; ArrayList<ShortcutInfo> children = info.contents; + setupContentForNumItems(children.size()); for (int i = 0; i < children.size(); i++) { ShortcutInfo child = (ShortcutInfo) children.get(i); - onAdd(child); + createAndAddShortcut(child); } mInfo.addListener(this); } @@ -460,25 +462,27 @@ public class Folder extends LinearLayout implements DragSource, OnItemLongClickL int countX = mContent.getCountX(); int countY = mContent.getCountY(); - if (countX * countY < count) { - // Current grid is too small, expand it - if (countX <= countY && countX < mMaxCountX) { - countX++; - } else if (countY < mMaxCountY) { - countY++; - } - if (countY == 0) countY++; - - mContent.setGridSize(countX, countY); - } else if ((countX - 1) * countY >= count || (countY - 1) * countX >= count) { - // Current grid is too big, shrink it - if (countX <= countY) { - countY--; - } else { - countX--; + boolean done = false; + + while (!done) { + int oldCountX = countX; + int oldCountY = countY; + if (countX * countY < count) { + // Current grid is too small, expand it + if (countX <= countY && countX < mMaxCountX) { + countX++; + } else if (countY < mMaxCountY) { + countY++; + } + if (countY == 0) countY++; + } else if ((countY - 1) * countX >= count && countY >= countX) { + countY = Math.max(0, countY - 1); + } else if ((countX - 1) * countY >= count) { + countX = Math.max(0, countX - 1); } - mContent.setGridSize(countX, countY); + done = countX == oldCountX && countY == oldCountY; } + mContent.setGridSize(countX, countY); arrangeChildren(list); } @@ -531,7 +535,6 @@ public class Folder extends LinearLayout implements DragSource, OnItemLongClickL } private void setupContentForNumItems(int count) { - setupContentDimension(count); CellLayout.LayoutParams lp = (CellLayout.LayoutParams) getLayoutParams(); @@ -561,10 +564,14 @@ public class Folder extends LinearLayout implements DragSource, OnItemLongClickL info.cellY = vacant[1]; boolean insert = false; mContent.addViewToCellLayout(v, insert ? 0 : -1, (int)info.id, lp, true); + LauncherModel.addOrMoveItemInDatabase(mLauncher, info, mInfo.id, 0, + info.cellX, info.cellY); } + mItemsInvalidated = true; } public void onAdd(ShortcutInfo item) { + mItemsInvalidated = true; if (!findAndSetEmptyCells(item)) { // The current layout is full, can we expand it? setupContentForNumItems(getItemCount() + 1); @@ -581,19 +588,6 @@ public class Folder extends LinearLayout implements DragSource, OnItemLongClickL return mContent.getChildrenLayout().getChildAt(index); } - private ArrayList<View> getItemsInReadingOrder() { - ArrayList<View> list = new ArrayList<View>(); - for (int j = 0; j < mContent.getCountY(); j++) { - for (int i = 0; i < mContent.getCountX(); i++) { - View v = mContent.getChildAt(i, j); - if (v != null) { - list.add(v); - } - } - } - return list; - } - private void onCloseComplete() { if (mRearrangeOnClose) { setupContentForNumItems(getItemCount()); @@ -602,6 +596,7 @@ public class Folder extends LinearLayout implements DragSource, OnItemLongClickL } public void onRemove(ShortcutInfo item) { + mItemsInvalidated = true; View v = mContent.getChildAt(mDragItemPosition[0], mDragItemPosition[1]); mContent.removeView(v); if (mState == STATE_ANIMATING) { @@ -610,4 +605,20 @@ public class Folder extends LinearLayout implements DragSource, OnItemLongClickL setupContentForNumItems(getItemCount()); } } + + public ArrayList<View> getItemsInReadingOrder() { + if (mItemsInvalidated) { + mItemsInReadingOrder.clear(); + for (int j = 0; j < mContent.getCountY(); j++) { + for (int i = 0; i < mContent.getCountX(); i++) { + View v = mContent.getChildAt(i, j); + if (v != null) { + mItemsInReadingOrder.add(v); + } + } + } + mItemsInvalidated = false; + } + return mItemsInReadingOrder; + } } |