Landscape UI tweaks based upon UX/product feedback

- Show preview images but reduce height.
- Reduce top margin above icons in target list.

Bug: 267522604
Test: Manual test with a variety of inputs from ShareTest in portrait
      and landscape.
Change-Id: I4a373b5c9cab555ffb4c28447f76e057d4114902

1 files changed, 7 insertions, 13 deletions

diff --git a/java/src/com/android/intentresolver/contentpreview/UnifiedContentPreviewUi.java b/java/src/com/android/intentresolver/contentpreview/UnifiedContentPreviewUi.java
index 04c1664b..c918a7f6 100644
--- a/java/src/com/android/intentresolver/contentpreview/UnifiedContentPreviewUi.java
+++ b/java/src/com/android/intentresolver/contentpreview/UnifiedContentPreviewUi.java
@@ -81,7 +81,6 @@ class UnifiedContentPreviewUi extends ContentPreviewUi {
                 R.layout.chooser_grid_preview_image, parent, false);
         ScrollableImagePreviewView imagePreview =
                 contentPreviewLayout.findViewById(R.id.scrollable_image_preview);
-        boolean showImages = !parent.getContext().getResources().getBoolean(R.bool.minimal_preview);
 
         final ActionRow actionRow =
                 contentPreviewLayout.findViewById(com.android.internal.R.id.chooser_action_row);
@@ -113,24 +112,19 @@ class UnifiedContentPreviewUi extends ContentPreviewUi {
             allImages = allImages && previewType == ScrollableImagePreviewView.PreviewType.Image;
             allVideos = allVideos && previewType == ScrollableImagePreviewView.PreviewType.Video;
 
-            if (showImages && fileInfo.getPreviewUri() != null) {
+            if (fileInfo.getPreviewUri() != null) {
                 previews.add(new ScrollableImagePreviewView.Preview(
                         previewType,
                         fileInfo.getPreviewUri()));
             }
         }
 
-        if (showImages) {
-            imagePreview.setOnNoPreviewCallback(() -> imagePreview.setVisibility(View.GONE));
-            imagePreview.setTransitionElementStatusCallback(mTransitionElementStatusCallback);
-            imagePreview.setPreviews(
-                    previews,
-                    mFiles.size() - previews.size(),
-                    mImageLoader);
-        } else {
-            imagePreview.setVisibility(View.GONE);
-            mTransitionElementStatusCallback.onAllTransitionElementsReady();
-        }
+        imagePreview.setOnNoPreviewCallback(() -> imagePreview.setVisibility(View.GONE));
+        imagePreview.setTransitionElementStatusCallback(mTransitionElementStatusCallback);
+        imagePreview.setPreviews(
+                previews,
+                mFiles.size() - previews.size(),
+                mImageLoader);
 
         if (allImages) {
             displayHeadline(