From 9be13fd2abe9ae7dbf521ab2f7ce4c29bfb44b96 Mon Sep 17 00:00:00 2001
From: Christine Emrich <christine.emrich@web.de>
Date: Sun, 27 Aug 2017 17:15:34 +0200
Subject: [PATCH 1/4] Center screenshots when they don't fill screen width

---
 .../res/layout/app_details2_screenshots.xml    | 18 ++++++++++++++----
 1 file changed, 14 insertions(+), 4 deletions(-)

diff --git a/app/src/main/res/layout/app_details2_screenshots.xml b/app/src/main/res/layout/app_details2_screenshots.xml
index 4d8d98c7a..c0b33c601 100644
--- a/app/src/main/res/layout/app_details2_screenshots.xml
+++ b/app/src/main/res/layout/app_details2_screenshots.xml
@@ -1,7 +1,17 @@
 <?xml version="1.0" encoding="utf-8"?>
-<android.support.v7.widget.RecyclerView
-    xmlns:android="http://schemas.android.com/apk/res/android"
-    android:id="@+id/screenshots"
+<FrameLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:app="http://schemas.android.com/apk/res-auto"
+    xmlns:tools="http://schemas.android.com/tools"
     android:layout_width="match_parent"
     android:layout_height="@dimen/details_screenshot_height">
-</android.support.v7.widget.RecyclerView>
\ No newline at end of file
+
+    <android.support.v7.widget.RecyclerView
+        android:id="@+id/screenshots"
+        android:layout_width="wrap_content"
+        android:layout_height="match_parent"
+        android:layout_gravity="center"
+        android:orientation="horizontal"
+        app:layoutManager="LinearLayoutManager"
+        tools:listitem="@layout/app_details2_screenshot_item" />
+
+</FrameLayout>
\ No newline at end of file

From e9cdc5e952b9dd198e0e35c60f0da2580bcb83eb Mon Sep 17 00:00:00 2001
From: Christine Emrich <christine.emrich@web.de>
Date: Sun, 27 Aug 2017 17:24:53 +0200
Subject: [PATCH 2/4] Remove highlight from snapped screenshot #1068

---
 .../views/AppDetailsRecyclerViewAdapter.java  |  1 -
 .../views/ScreenShotsRecyclerViewAdapter.java | 43 +------------------
 app/src/main/res/values/dimens.xml            |  4 --
 3 files changed, 2 insertions(+), 46 deletions(-)

diff --git a/app/src/main/java/org/fdroid/fdroid/views/AppDetailsRecyclerViewAdapter.java b/app/src/main/java/org/fdroid/fdroid/views/AppDetailsRecyclerViewAdapter.java
index 1ddb1fd80..2d82805db 100644
--- a/app/src/main/java/org/fdroid/fdroid/views/AppDetailsRecyclerViewAdapter.java
+++ b/app/src/main/java/org/fdroid/fdroid/views/AppDetailsRecyclerViewAdapter.java
@@ -582,7 +582,6 @@ public class AppDetailsRecyclerViewAdapter
                 snapHelper.attachToRecyclerView(null);
             }
             snapHelper = new LinearLayoutManagerSnapHelper(lm);
-            snapHelper.setLinearSnapHelperListener(adapter);
             snapHelper.attachToRecyclerView(recyclerView);
         }
     }
diff --git a/app/src/main/java/org/fdroid/fdroid/views/ScreenShotsRecyclerViewAdapter.java b/app/src/main/java/org/fdroid/fdroid/views/ScreenShotsRecyclerViewAdapter.java
index e82b36d46..048395dfe 100644
--- a/app/src/main/java/org/fdroid/fdroid/views/ScreenShotsRecyclerViewAdapter.java
+++ b/app/src/main/java/org/fdroid/fdroid/views/ScreenShotsRecyclerViewAdapter.java
@@ -1,9 +1,7 @@
 package org.fdroid.fdroid.views;
 
 import android.content.Context;
-import android.content.res.Resources;
 import android.graphics.Bitmap;
-import android.support.v4.view.ViewCompat;
 import android.support.v7.widget.RecyclerView;
 import android.view.LayoutInflater;
 import android.view.View;
@@ -17,22 +15,13 @@ import com.nostra13.universalimageloader.core.assist.ImageScaleType;
 import org.fdroid.fdroid.R;
 import org.fdroid.fdroid.data.App;
 
-public class ScreenShotsRecyclerViewAdapter extends RecyclerView.Adapter<RecyclerView.ViewHolder>
-        implements LinearLayoutManagerSnapHelper.LinearSnapHelperListener {
+class ScreenShotsRecyclerViewAdapter extends RecyclerView.Adapter<RecyclerView.ViewHolder> {
     private final String[] screenshots;
     private final DisplayImageOptions displayImageOptions;
-    private View selectedView;
-    private int selectedPosition;
-    private final int selectedItemElevation;
-    private final int unselectedItemMargin;
 
-    public ScreenShotsRecyclerViewAdapter(Context context, App app) {
+    ScreenShotsRecyclerViewAdapter(Context context, App app) {
         super();
         screenshots = app.getAllScreenshots(context);
-        selectedPosition = 0;
-        Resources r = context.getResources();
-        selectedItemElevation = r.getDimensionPixelSize(R.dimen.details_screenshot_selected_elevation);
-        unselectedItemMargin = r.getDimensionPixelSize(R.dimen.details_screenshot_margin);
         displayImageOptions = new DisplayImageOptions.Builder()
                 .cacheInMemory(true)
                 .cacheOnDisk(true)
@@ -46,10 +35,6 @@ public class ScreenShotsRecyclerViewAdapter extends RecyclerView.Adapter<Recycle
     @Override
     public void onBindViewHolder(RecyclerView.ViewHolder holder, int position) {
         ScreenShotViewHolder vh = (ScreenShotViewHolder) holder;
-        setViewSelected(vh.itemView, position == selectedPosition);
-        if (position == selectedPosition) {
-            this.selectedView = vh.itemView;
-        }
         ImageLoader.getInstance().displayImage(screenshots[position],
                 vh.image, displayImageOptions);
     }
@@ -66,30 +51,6 @@ public class ScreenShotsRecyclerViewAdapter extends RecyclerView.Adapter<Recycle
         return screenshots.length;
     }
 
-    @Override
-    public void onSnappedToView(View view, int snappedPosition) {
-        // Deselect the previous selected view first
-        setViewSelected(selectedView, false);
-
-        // Change the selected view to the newly snapped-to view.
-        selectedView = view;
-        selectedPosition = snappedPosition;
-        setViewSelected(selectedView, true);
-    }
-
-    private void setViewSelected(View view, boolean selected) {
-        if (view != null) {
-            RecyclerView.LayoutParams lp = (RecyclerView.LayoutParams) view.getLayoutParams();
-            if (selected) {
-                lp.setMargins(0, selectedItemElevation, 0, selectedItemElevation);
-            } else {
-                lp.setMargins(0, unselectedItemMargin, 0, unselectedItemMargin);
-            }
-            ViewCompat.setElevation(view, selected ? selectedItemElevation : selectedItemElevation / 2);
-            view.setLayoutParams(lp);
-        }
-    }
-
     private class ScreenShotViewHolder extends RecyclerView.ViewHolder {
         final ImageView image;
 
diff --git a/app/src/main/res/values/dimens.xml b/app/src/main/res/values/dimens.xml
index 054dd03be..93a768f60 100644
--- a/app/src/main/res/values/dimens.xml
+++ b/app/src/main/res/values/dimens.xml
@@ -13,10 +13,6 @@
 
     <dimen name="details_screenshot_height">200dp</dimen>
     <dimen name="details_screenshot_width">200dp</dimen>
-    <!-- "Not selected" items are inset by this value, while the selected one is not -->
-    <dimen name="details_screenshot_margin">8dp</dimen>
-    <!-- The selected item stands out from the background by this elevation -->
-    <dimen name="details_screenshot_selected_elevation">3dp</dimen>
 
     <dimen name="fab_margin">16dp</dimen>
 

From e77281b99b216b2663b92a2a6122d8a70b8f724a Mon Sep 17 00:00:00 2001
From: Christine Emrich <christine.emrich@web.de>
Date: Sun, 27 Aug 2017 17:32:49 +0200
Subject: [PATCH 3/4] Give screenshots some margin

This way the card views elevation can be displayed
properly and multiple screenshots don't merge together.
---
 .../res/layout/app_details2_screenshot_item.xml     | 13 +++++++------
 app/src/main/res/values/dimens.xml                  |  1 +
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/app/src/main/res/layout/app_details2_screenshot_item.xml b/app/src/main/res/layout/app_details2_screenshot_item.xml
index 114dc2269..ce727453f 100644
--- a/app/src/main/res/layout/app_details2_screenshot_item.xml
+++ b/app/src/main/res/layout/app_details2_screenshot_item.xml
@@ -1,18 +1,19 @@
 <?xml version="1.0" encoding="utf-8"?>
 <android.support.v7.widget.CardView xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:app="http://schemas.android.com/apk/res-auto"
     android:layout_width="wrap_content"
     android:layout_height="match_parent"
-    xmlns:app="http://schemas.android.com/apk/res-auto"
-    app:cardElevation="3dp"
-    app:cardBackgroundColor="#ffffff"
+    android:layout_margin="@dimen/details_screenshot_margin"
     android:padding="10dp"
-    >
+    app:cardBackgroundColor="#ffffff"
+    app:cardElevation="3dp">
+
     <ImageView
         android:id="@+id/image"
         android:layout_width="wrap_content"
         android:layout_height="match_parent"
         android:adjustViewBounds="true"
         android:minWidth="@dimen/details_screenshot_width"
-        android:scaleType="fitCenter"
-        />
+        android:scaleType="fitCenter" />
+
 </android.support.v7.widget.CardView>
\ No newline at end of file
diff --git a/app/src/main/res/values/dimens.xml b/app/src/main/res/values/dimens.xml
index 93a768f60..dc3213c5f 100644
--- a/app/src/main/res/values/dimens.xml
+++ b/app/src/main/res/values/dimens.xml
@@ -13,6 +13,7 @@
 
     <dimen name="details_screenshot_height">200dp</dimen>
     <dimen name="details_screenshot_width">200dp</dimen>
+    <dimen name="details_screenshot_margin">4dp</dimen>
 
     <dimen name="fab_margin">16dp</dimen>
 

From 325060300055be092f3041fc0ff66df4213c34a9 Mon Sep 17 00:00:00 2001
From: Christine Emrich <christine.emrich@web.de>
Date: Sun, 27 Aug 2017 17:40:48 +0200
Subject: [PATCH 4/4] Make screenshots bigger

---
 app/src/main/res/values/dimens.xml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/src/main/res/values/dimens.xml b/app/src/main/res/values/dimens.xml
index dc3213c5f..91a41362a 100644
--- a/app/src/main/res/values/dimens.xml
+++ b/app/src/main/res/values/dimens.xml
@@ -11,8 +11,8 @@
          applied. The collapsable lists have a little extra on the sides. -->
     <dimen name="details_activity__collapsable_lists__padding__horizontal">4dp</dimen>
 
-    <dimen name="details_screenshot_height">200dp</dimen>
-    <dimen name="details_screenshot_width">200dp</dimen>
+    <dimen name="details_screenshot_height">240dp</dimen>
+    <dimen name="details_screenshot_width">240dp</dimen>
     <dimen name="details_screenshot_margin">4dp</dimen>
 
     <dimen name="fab_margin">16dp</dimen>