diff --git a/packages/PrintSpooler/res/drawable-hdpi/ic_check_circle.png b/packages/PrintSpooler/res/drawable-hdpi/ic_check_circle.png deleted file mode 100644 index 4ad541771f9fb..0000000000000 Binary files a/packages/PrintSpooler/res/drawable-hdpi/ic_check_circle.png and /dev/null differ diff --git a/packages/PrintSpooler/res/drawable-hdpi/ic_remove_circle.png b/packages/PrintSpooler/res/drawable-hdpi/ic_remove_circle.png deleted file mode 100644 index ef053b6f1be61..0000000000000 Binary files a/packages/PrintSpooler/res/drawable-hdpi/ic_remove_circle.png and /dev/null differ diff --git a/packages/PrintSpooler/res/drawable-mdpi/ic_check_circle.png b/packages/PrintSpooler/res/drawable-mdpi/ic_check_circle.png deleted file mode 100644 index f66065affeedc..0000000000000 Binary files a/packages/PrintSpooler/res/drawable-mdpi/ic_check_circle.png and /dev/null differ diff --git a/packages/PrintSpooler/res/drawable-mdpi/ic_remove_circle.png b/packages/PrintSpooler/res/drawable-mdpi/ic_remove_circle.png deleted file mode 100644 index 7e044ac553a13..0000000000000 Binary files a/packages/PrintSpooler/res/drawable-mdpi/ic_remove_circle.png and /dev/null differ diff --git a/packages/PrintSpooler/res/drawable-xhdpi/ic_check_circle.png b/packages/PrintSpooler/res/drawable-xhdpi/ic_check_circle.png deleted file mode 100644 index d8ea4d2915873..0000000000000 Binary files a/packages/PrintSpooler/res/drawable-xhdpi/ic_check_circle.png and /dev/null differ diff --git a/packages/PrintSpooler/res/drawable-xhdpi/ic_remove_circle.png b/packages/PrintSpooler/res/drawable-xhdpi/ic_remove_circle.png deleted file mode 100644 index 622989c7d6f1e..0000000000000 Binary files a/packages/PrintSpooler/res/drawable-xhdpi/ic_remove_circle.png and /dev/null differ diff --git a/packages/PrintSpooler/res/drawable-xxhdpi/ic_check_circle.png b/packages/PrintSpooler/res/drawable-xxhdpi/ic_check_circle.png deleted file mode 100644 index ac36eba5dc4cb..0000000000000 Binary files a/packages/PrintSpooler/res/drawable-xxhdpi/ic_check_circle.png and /dev/null differ diff --git a/packages/PrintSpooler/res/drawable-xxhdpi/ic_remove_circle.png b/packages/PrintSpooler/res/drawable-xxhdpi/ic_remove_circle.png deleted file mode 100644 index 303ccfb0a76c3..0000000000000 Binary files a/packages/PrintSpooler/res/drawable-xxhdpi/ic_remove_circle.png and /dev/null differ diff --git a/packages/PrintSpooler/res/drawable-xxxhdpi/ic_check_circle.png b/packages/PrintSpooler/res/drawable-xxxhdpi/ic_check_circle.png deleted file mode 100644 index 1737f0a765aec..0000000000000 Binary files a/packages/PrintSpooler/res/drawable-xxxhdpi/ic_check_circle.png and /dev/null differ diff --git a/packages/PrintSpooler/res/drawable-xxxhdpi/ic_remove_circle.png b/packages/PrintSpooler/res/drawable-xxxhdpi/ic_remove_circle.png deleted file mode 100644 index e9c6252951524..0000000000000 Binary files a/packages/PrintSpooler/res/drawable-xxxhdpi/ic_remove_circle.png and /dev/null differ diff --git a/packages/PrintSpooler/res/drawable/ic_check_circle.xml b/packages/PrintSpooler/res/drawable/ic_check_circle.xml new file mode 100644 index 0000000000000..0a07b1a4e91ea --- /dev/null +++ b/packages/PrintSpooler/res/drawable/ic_check_circle.xml @@ -0,0 +1,25 @@ + + + + + + diff --git a/packages/PrintSpooler/res/drawable/ic_remove_circle.xml b/packages/PrintSpooler/res/drawable/ic_remove_circle.xml new file mode 100644 index 0000000000000..50ce2e88a68ad --- /dev/null +++ b/packages/PrintSpooler/res/drawable/ic_remove_circle.xml @@ -0,0 +1,25 @@ + + + + + + diff --git a/packages/PrintSpooler/res/drawable/page_selector_background.xml b/packages/PrintSpooler/res/drawable/page_selector_background.xml deleted file mode 100644 index 4d32328efdc90..0000000000000 --- a/packages/PrintSpooler/res/drawable/page_selector_background.xml +++ /dev/null @@ -1,36 +0,0 @@ - - - - - - - - - - - - - - - - diff --git a/packages/PrintSpooler/res/layout/preview_page.xml b/packages/PrintSpooler/res/layout/preview_page.xml index edce28965001c..aafdd8fc035c1 100644 --- a/packages/PrintSpooler/res/layout/preview_page.xml +++ b/packages/PrintSpooler/res/layout/preview_page.xml @@ -20,12 +20,14 @@ android:layout_height="wrap_content" android:layout_margin="@dimen/preview_page_margin" android:orientation="vertical" - android:background="?android:attr/colorForeground"> + android:background="@color/unselected_page_background_color" + android:elevation="@dimen/unselected_page_elevation" > + android:layout_height="fill_parent" + android:alpha="0.5"> + android:alpha="0.5"> + android:src="@drawable/ic_remove_circle"> diff --git a/packages/PrintSpooler/res/layout/preview_page_selected.xml b/packages/PrintSpooler/res/layout/preview_page_selected.xml new file mode 100644 index 0000000000000..77f4727434e69 --- /dev/null +++ b/packages/PrintSpooler/res/layout/preview_page_selected.xml @@ -0,0 +1,60 @@ + + + + + + + + + + + + + + + + + + + diff --git a/packages/PrintSpooler/res/values/colors.xml b/packages/PrintSpooler/res/values/colors.xml index 47e616ef40c77..9b4e94404c70d 100644 --- a/packages/PrintSpooler/res/values/colors.xml +++ b/packages/PrintSpooler/res/values/colors.xml @@ -22,6 +22,8 @@ #F2F1F2 + #C0C0C0 + #ffa3a3a3 diff --git a/packages/PrintSpooler/res/values/constants.xml b/packages/PrintSpooler/res/values/constants.xml index f6c52d3c04dc6..54a819f3ada75 100644 --- a/packages/PrintSpooler/res/values/constants.xml +++ b/packages/PrintSpooler/res/values/constants.xml @@ -39,9 +39,6 @@ 2 - 100% - 50% - 32dip 128dip diff --git a/packages/PrintSpooler/src/com/android/printspooler/ui/PageAdapter.java b/packages/PrintSpooler/src/com/android/printspooler/ui/PageAdapter.java index c1a3f865bed46..54400b3b85405 100644 --- a/packages/PrintSpooler/src/com/android/printspooler/ui/PageAdapter.java +++ b/packages/PrintSpooler/src/com/android/printspooler/ui/PageAdapter.java @@ -297,7 +297,14 @@ public final class PageAdapter extends Adapter { @Override public ViewHolder onCreateViewHolder(ViewGroup parent, int viewType) { - View page = mLayoutInflater.inflate(R.layout.preview_page, parent, false); + View page; + + if (viewType == 0) { + page = mLayoutInflater.inflate(R.layout.preview_page_selected, parent, false); + } else { + page = mLayoutInflater.inflate(R.layout.preview_page, parent, false); + } + return new MyViewHolder(page); } @@ -345,9 +352,9 @@ public final class PageAdapter extends Adapter { content.init(provider, mEmptyState, mErrorState, mMediaSize, mMinMargins); if (mConfirmedPagesInDocument.indexOfKey(pageInDocument) >= 0) { - page.setSelected(true, false); + page.setSelected(true); } else { - page.setSelected(false, false); + page.setSelected(false); } page.setContentDescription(mContext.getString(R.string.page_description_template, @@ -364,6 +371,15 @@ public final class PageAdapter extends Adapter { return mSelectedPageCount; } + @Override + public int getItemViewType(int position) { + if (mConfirmedPagesInDocument.indexOfKey(computePageIndexInDocument(position)) >= 0) { + return 0; + } else { + return 1; + } + } + @Override public long getItemId(int position) { return computePageIndexInDocument(position); @@ -821,14 +837,14 @@ public final class PageAdapter extends Adapter { final int pageInDocument = computePageIndexInDocument(pageInAdapter); if (mConfirmedPagesInDocument.indexOfKey(pageInDocument) < 0) { mConfirmedPagesInDocument.put(pageInDocument, null); - page.setSelected(true, true); } else { if (mConfirmedPagesInDocument.size() <= 1) { return; } mConfirmedPagesInDocument.remove(pageInDocument); - page.setSelected(false, true); } + + notifyItemChanged(pageInAdapter); } } } diff --git a/packages/PrintSpooler/src/com/android/printspooler/widget/PreviewPageFrame.java b/packages/PrintSpooler/src/com/android/printspooler/widget/PreviewPageFrame.java index 6a6f1d3957da3..95bdb093aafe7 100644 --- a/packages/PrintSpooler/src/com/android/printspooler/widget/PreviewPageFrame.java +++ b/packages/PrintSpooler/src/com/android/printspooler/widget/PreviewPageFrame.java @@ -22,29 +22,14 @@ import android.view.accessibility.AccessibilityEvent; import android.view.accessibility.AccessibilityNodeInfo; import android.widget.CompoundButton; import android.widget.LinearLayout; -import com.android.printspooler.R; /** * This class represents the frame of page in the print preview list * that contains the page and a footer. */ public final class PreviewPageFrame extends LinearLayout { - private final float mSelectedElevation; - private final float mNotSelectedElevation; - - private final float mSelectedPageAlpha; - private final float mNotSelectedAlpha; - public PreviewPageFrame(Context context, AttributeSet attrs) { super(context, attrs); - mSelectedElevation = mContext.getResources().getDimension( - R.dimen.selected_page_elevation); - mNotSelectedElevation = mContext.getResources().getDimension( - R.dimen.unselected_page_elevation); - mSelectedPageAlpha = mContext.getResources().getFraction( - R.fraction.page_selected_alpha, 1, 1); - mNotSelectedAlpha = mContext.getResources().getFraction( - R.fraction.page_unselected_alpha, 1, 1); } @Override @@ -65,28 +50,4 @@ public final class PreviewPageFrame extends LinearLayout { info.setCheckable(true); info.setChecked(isSelected()); } - - public void setSelected(boolean selected, boolean animate) { - if (isSelected() == selected) { - return; - } - setSelected(selected); - if (selected) { - if (animate) { - animate().translationZ(mSelectedElevation) - .alpha(mSelectedPageAlpha); - } else { - setTranslationZ(mSelectedElevation); - setAlpha(mSelectedPageAlpha); - } - } else { - if (animate) { - animate().translationZ(mNotSelectedElevation) - .alpha(mNotSelectedAlpha); - } else { - setTranslationZ(mNotSelectedElevation); - setAlpha(mNotSelectedAlpha); - } - } - } }