From 72e9bda3d304540515742135bf12e5f1dbed8857 Mon Sep 17 00:00:00 2001 From: 1 Date: Mon, 8 May 2023 21:01:33 +0000 Subject: Fixes based upon UX feedback - Align colors with mocks. - Increase custom action vertical padding - Align text + image and file + text to the same 56x56 image size - Reduce target vertical padding - Don't disable text when share is "image only" - Add an icon for single-file shares - Update multi-file icon to align with mocks. - Vertically align image in text+image - Show image and title with text in landscape - Reduce text preview padding to align with file+text padding. Bug: 281541996 Test: build and test with ShareTest, Files and Photos. Change-Id: Ia58315d765a824cb45ee1b84625c1262a8a3e76d --- .../src/com/android/intentresolver/UnbundledChooserActivityTest.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'java/tests/src') diff --git a/java/tests/src/com/android/intentresolver/UnbundledChooserActivityTest.java b/java/tests/src/com/android/intentresolver/UnbundledChooserActivityTest.java index 5ea0b47d..317e2815 100644 --- a/java/tests/src/com/android/intentresolver/UnbundledChooserActivityTest.java +++ b/java/tests/src/com/android/intentresolver/UnbundledChooserActivityTest.java @@ -26,7 +26,6 @@ import static androidx.test.espresso.assertion.ViewAssertions.doesNotExist; import static androidx.test.espresso.assertion.ViewAssertions.matches; import static androidx.test.espresso.matcher.ViewMatchers.hasSibling; import static androidx.test.espresso.matcher.ViewMatchers.isDisplayed; -import static androidx.test.espresso.matcher.ViewMatchers.isEnabled; import static androidx.test.espresso.matcher.ViewMatchers.withEffectiveVisibility; import static androidx.test.espresso.matcher.ViewMatchers.withId; import static androidx.test.espresso.matcher.ViewMatchers.withText; @@ -855,7 +854,7 @@ public class UnbundledChooserActivityTest { onView(withId(R.id.image_view)) .check(matches(withEffectiveVisibility(ViewMatchers.Visibility.GONE))); onView(withId(R.id.content_preview_text)) - .check(matches(allOf(isDisplayed(), not(isEnabled()), withText("Image only")))); + .check(matches(allOf(isDisplayed(), withText("Image only")))); } @Test -- cgit v1.2.3-59-g8ed1b