Update Search UX for OC

- Change text opacity for Search UX
- Update dps between summary and breadcrumbs
- Remove icons for Title items
- Restrict summary to single line

Test: manual
Bug: 36853763
Change-Id: I2acd574ec9c490bbe7097d30c512e58ebe1ad9f2
This commit is contained in:
Matthew Fritze
2017-03-31 17:07:25 -07:00
parent 008a9adef9
commit d0b42b0322
6 changed files with 8 additions and 9 deletions

View File

@@ -20,7 +20,7 @@
android:id="@+id/breadcrumb" android:id="@+id/breadcrumb"
android:layout_width="wrap_content" android:layout_width="wrap_content"
android:layout_height="wrap_content" android:layout_height="wrap_content"
android:paddingTop="8dp" android:paddingTop="5dp"
android:textAppearance="?android:attr/textAppearanceListItemSecondary" android:textAppearance="?android:attr/textAppearanceListItemSecondary"
android:textColor="?android:attr/textColorSecondary" android:textColor="?android:attr/textColorSecondary"
android:ellipsize="marquee"/> android:ellipsize="marquee"/>

View File

@@ -46,9 +46,8 @@
android:id="@android:id/summary" android:id="@android:id/summary"
android:layout_width="wrap_content" android:layout_width="wrap_content"
android:layout_height="wrap_content" android:layout_height="wrap_content"
android:textAppearance="?android:attr/textAppearanceListItemSecondary" android:singleLine="true"
android:textColor="?android:attr/textColorSecondary" android:textAppearance="?android:attr/textAppearanceListItemSecondary"/>
android:maxLines="10"/>
<include layout="@layout/search_breadcrumb_view"/> <include layout="@layout/search_breadcrumb_view"/>
</LinearLayout> </LinearLayout>

View File

@@ -46,9 +46,8 @@
android:id="@android:id/summary" android:id="@android:id/summary"
android:layout_width="wrap_content" android:layout_width="wrap_content"
android:layout_height="wrap_content" android:layout_height="wrap_content"
android:singleLine="true"
android:textAppearance="?android:attr/textAppearanceListItemSecondary" android:textAppearance="?android:attr/textAppearanceListItemSecondary"
android:textColor="?android:attr/textColorSecondary"
android:maxLines="3"
android:ellipsize="marquee"/> android:ellipsize="marquee"/>
<include layout="@layout/search_breadcrumb_view"/> <include layout="@layout/search_breadcrumb_view"/>

View File

@@ -749,7 +749,6 @@ public class DatabaseIndexingManager {
.setEntries(null) .setEntries(null)
.setClassName(fragmentName) .setClassName(fragmentName)
.setScreenTitle(screenTitle) .setScreenTitle(screenTitle)
.setIconResId(iconResId)
.setRank(rank) .setRank(rank)
.setIntentAction(intentAction) .setIntentAction(intentAction)
.setIntentTargetPackage(intentTargetPackage) .setIntentTargetPackage(intentTargetPackage)

View File

@@ -66,9 +66,11 @@ public abstract class SearchViewHolder extends RecyclerView.ViewHolder {
iconView.setImageDrawable(appResult.info.loadIcon(pm)); iconView.setImageDrawable(appResult.info.loadIcon(pm));
} else if (result.icon != null) { } else if (result.icon != null) {
iconView.setImageDrawable(result.icon); iconView.setImageDrawable(result.icon);
// TODO set color of icon
} else { } else {
iconView.setBackgroundResource(R.drawable.empty_icon); iconView.setBackgroundResource(R.drawable.empty_icon);
} }
bindBreadcrumbView(result); bindBreadcrumbView(result);
} }

View File

@@ -303,7 +303,7 @@ public class DatabaseIndexingManagerTest {
// Class Name // Class Name
assertThat(cursor.getString(11)).isEqualTo(className); assertThat(cursor.getString(11)).isEqualTo(className);
// Icon // Icon
assertThat(cursor.getInt(12)).isEqualTo(iconResId); assertThat(cursor.getInt(12)).isEqualTo(0);
// Intent Action // Intent Action
assertThat(cursor.getString(13)).isEqualTo(action); assertThat(cursor.getString(13)).isEqualTo(action);
// Target Package // Target Package