Merge branch 'Gargron-master'

app/.gitignore

@@ -1,2 +1,3 @@
 /build
-app-release.apk
+app-release.apk
+google-services.json

app/src/main/java/com/keylesspalace/tusky/ConversationLineItemDecoration.java

@@ -21,8 +21,6 @@ class ConversationLineItemDecoration extends RecyclerView.ItemDecoration {
 
     @Override
     public void onDraw(Canvas c, RecyclerView parent, RecyclerView.State state) {
-        // Fun fact: this method draws in pixels, but all layouts are in DP, so I'm using the divider's
-        // own 2dp width to calculate what I want
         int dividerLeft  = parent.getPaddingLeft() + mContext.getResources().getDimensionPixelSize(R.dimen.status_left_line_margin);
         int dividerRight = dividerLeft + mDivider.getIntrinsicWidth();
 

app/src/main/java/com/keylesspalace/tusky/LoginActivity.java

@@ -39,6 +39,8 @@ import java.util.Map;
 import retrofit2.Call;
 import retrofit2.Callback;
 import retrofit2.Response;
+import retrofit2.Retrofit;
+import retrofit2.converter.gson.GsonConverterFactory;
 
 public class LoginActivity extends BaseActivity {
     private static String OAUTH_SCOPES = "read write follow";
@@ -94,6 +96,15 @@ public class LoginActivity extends BaseActivity {
         startActivity(viewIntent);
     }
 
+    private MastodonAPI getApiFor(String domain) {
+        Retrofit retrofit = new Retrofit.Builder()
+                .baseUrl("https://" + domain)
+                .addConverterFactory(GsonConverterFactory.create())
+                .build();
+
+        return retrofit.create(MastodonAPI.class);
+    }
+
     /**
      * Obtain the oauth client credentials for this app. This is only necessary the first time the
      * app is run on a given server instance. So, after the first authentication, they are
@@ -134,9 +145,7 @@ public class LoginActivity extends BaseActivity {
                 }
             };
 
-            List<String> redirectUris = new ArrayList<>();
+            getApiFor(domain).authenticateApp(getString(R.string.app_name), getOauthRedirectUri(), OAUTH_SCOPES,
-            redirectUris.add(getOauthRedirectUri());
-            mastodonAPI.authenticateApp(getString(R.string.app_name), redirectUris, OAUTH_SCOPES,
                     getString(R.string.app_website)).enqueue(callback);
 
         }
@@ -245,7 +254,7 @@ public class LoginActivity extends BaseActivity {
                         editText.setError(t.getMessage());
                     }
                 };
-                mastodonAPI.fetchOAuthToken(clientId, clientSecret, redirectUri, code,
+                getApiFor(domain).fetchOAuthToken(clientId, clientSecret, redirectUri, code,
                         "authorization_code").enqueue(callback);
             } else if (error != null) {
                 /* Authorization failed. Put the error response where the user can read it and they

app/src/main/java/com/keylesspalace/tusky/MastodonAPI.java

@@ -175,7 +175,7 @@ public interface MastodonAPI {
     @POST("api/v1/apps")
     Call<AppCredentials> authenticateApp(
             @Field("client_name") String clientName,
-            @Field("redirect_uris[]") List<String> redirectUris,
+            @Field("redirect_uris") String redirectUris,
             @Field("scopes") String scopes,
             @Field("website") String website);
 

app/src/main/java/com/keylesspalace/tusky/MyFirebaseMessagingService.java

@@ -160,6 +160,7 @@ public class MyFirebaseMessagingService extends FirebaseMessagingService {
         Picasso.with(this)
                 .load(body.account.avatar)
                 .placeholder(R.drawable.avatar_default)
+                .transform(new RoundedTransformation(7, 0))
                 .into(mTarget);
 
         if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP) {

app/src/main/java/com/keylesspalace/tusky/RoundedTransformation.java

@@ -0,0 +1,45 @@
+package com.keylesspalace.tusky;
+
+import android.graphics.Bitmap;
+import android.graphics.BitmapShader;
+import android.graphics.Canvas;
+import android.graphics.Paint;
+import android.graphics.RectF;
+import android.graphics.Shader;
+
+import com.squareup.picasso.Transformation;
+
+public class RoundedTransformation implements Transformation {
+
+    private final int radius;
+    private final int margin;
+
+    public RoundedTransformation(final int radius, final int margin) {
+        this.radius = radius;
+        this.margin = margin;
+    }
+
+    @Override
+    public Bitmap transform(Bitmap source) {
+        final Paint paint = new Paint();
+
+        paint.setAntiAlias(true);
+        paint.setShader(new BitmapShader(source, Shader.TileMode.CLAMP, Shader.TileMode.CLAMP));
+
+        Bitmap output = Bitmap.createBitmap(source.getWidth(), source.getHeight(), Bitmap.Config.ARGB_8888);
+        Canvas canvas = new Canvas(output);
+
+        canvas.drawRoundRect(new RectF(margin, margin, source.getWidth() - margin, source.getHeight() - margin), radius, radius, paint);
+
+        if (source != output) {
+            source.recycle();
+        }
+
+        return output;
+    }
+
+    @Override
+    public String key() {
+        return "rounded";
+    }
+}

app/src/main/java/com/keylesspalace/tusky/StatusViewHolder.java

@@ -158,6 +158,7 @@ class StatusViewHolder extends RecyclerView.ViewHolder {
                 .load(url)
                 .placeholder(R.drawable.avatar_default)
                 .error(R.drawable.avatar_error)
+                .transform(new RoundedTransformation(7, 0))
                 .into(avatar);
     }
 

app/src/main/res/layout/item_status.xml

@@ -40,7 +40,7 @@
         android:scaleType="fitCenter"
         android:id="@+id/status_avatar"
         android:layout_below="@+id/status_reblogged_bar"
-        android:layout_marginTop="10dp"
+        android:layout_marginTop="11dp"
         android:layout_marginRight="10dp" />
 
     <RelativeLayout