diff --git a/briar-android/res/layout/author_view.xml b/briar-android/res/layout/author_view.xml index 92c863c37..13f19f53e 100644 --- a/briar-android/res/layout/author_view.xml +++ b/briar-android/res/layout/author_view.xml @@ -1,6 +1,7 @@ @@ -10,7 +11,9 @@ android:layout_height="@dimen/listitem_picture_size" android:layout_centerVertical="true" android:layout_marginLeft="@dimen/listitem_horizontal_margin" - android:layout_marginStart="@dimen/listitem_horizontal_margin"/> + android:layout_marginStart="@dimen/listitem_horizontal_margin" + app:civ_border_width="@dimen/avatar_border_width" + app:civ_border_color="@color/briar_text_primary"/> @@ -9,14 +10,16 @@ + android:layout_height="@dimen/dropdown_picture_size" + android:layout_margin="@dimen/margin_small" + app:civ_border_width="@dimen/avatar_border_width" + app:civ_border_color="@color/briar_text_primary"/> @@ -19,7 +20,9 @@ android:layout_alignParentStart="true" android:layout_centerVertical="true" android:layout_marginLeft="@dimen/listitem_horizontal_margin" - android:layout_marginStart="@dimen/listitem_horizontal_margin"/> + android:layout_marginStart="@dimen/listitem_horizontal_margin" + app:civ_border_width="@dimen/avatar_border_width" + app:civ_border_color="@color/briar_text_primary"/> + app:civ_border_width="@dimen/avatar_border_width" + app:civ_border_color="@color/briar_text_primary"/> 1dp 16dp - 40dp 72dp - 56dp - + 40dp 32dp + 1dp diff --git a/briar-android/src/im/delight/android/identicons/AsymmetricIdenticon.java b/briar-android/src/im/delight/android/identicons/AsymmetricIdenticon.java index 888b234cc..fa5c1c35a 100644 --- a/briar-android/src/im/delight/android/identicons/AsymmetricIdenticon.java +++ b/briar-android/src/im/delight/android/identicons/AsymmetricIdenticon.java @@ -17,7 +17,6 @@ package im.delight.android.identicons; */ import android.content.Context; -import android.graphics.Color; import android.util.AttributeSet; import org.briarproject.api.crypto.CryptoComponent; @@ -73,11 +72,6 @@ public class AsymmetricIdenticon extends IdenticonView { protected boolean isCellVisible(int row, int column) { return getByte(3 + row * getColumnCount() + column) >= 0; } - - @Override - protected int getIconColor() { - return Color.rgb(getByte(0) + 128, getByte(1) + 128, getByte(2) + 128); - } }; } diff --git a/briar-android/src/im/delight/android/identicons/IdenticonBase.java b/briar-android/src/im/delight/android/identicons/IdenticonBase.java index 59c842c12..e50ac91a6 100644 --- a/briar-android/src/im/delight/android/identicons/IdenticonBase.java +++ b/briar-android/src/im/delight/android/identicons/IdenticonBase.java @@ -93,15 +93,15 @@ public abstract class IdenticonBase { abstract protected boolean isCellVisible(int row, int column); - abstract protected int getIconColor(); + protected int getIconColor() { + int r = getByte(0) * 3 / 4 + 96; + int g = getByte(1) * 3 / 4 + 96; + int b = getByte(2) * 3 / 4 + 96; + return Color.rgb(r, g, b); + } protected int getBackgroundColor() { - float[] hsv = new float[3]; - Color.colorToHSV(getIconColor(), hsv); - if (hsv[2] < 0.5) - return Color.parseColor("#ffeeeeee"); // @color/background_material_light - else - return Color.parseColor("#ff303030"); // @color/background_material_dark + return Color.WHITE; } public void updateSize(int w, int h) { diff --git a/briar-android/src/im/delight/android/identicons/IdenticonDrawable.java b/briar-android/src/im/delight/android/identicons/IdenticonDrawable.java index 135c1029e..8034a6a1d 100644 --- a/briar-android/src/im/delight/android/identicons/IdenticonDrawable.java +++ b/briar-android/src/im/delight/android/identicons/IdenticonDrawable.java @@ -1,7 +1,6 @@ package im.delight.android.identicons; import android.graphics.Canvas; -import android.graphics.Color; import android.graphics.ColorFilter; import android.graphics.Rect; import android.graphics.drawable.Drawable; @@ -39,11 +38,6 @@ public class IdenticonDrawable extends Drawable { protected boolean isCellVisible(int row, int column) { return getByte(3 + row * CENTER_COLUMN_INDEX + getSymmetricColumnIndex(column)) >= 0; } - - @Override - protected int getIconColor() { - return Color.rgb(getByte(0) + 128, getByte(1) + 128, getByte(2) + 128); - } }; mDelegate.show(toShow); } diff --git a/briar-android/src/im/delight/android/identicons/SymmetricIdenticon.java b/briar-android/src/im/delight/android/identicons/SymmetricIdenticon.java index 875cbe9ad..f39027081 100644 --- a/briar-android/src/im/delight/android/identicons/SymmetricIdenticon.java +++ b/briar-android/src/im/delight/android/identicons/SymmetricIdenticon.java @@ -17,7 +17,6 @@ package im.delight.android.identicons; */ import android.content.Context; -import android.graphics.Color; import android.util.AttributeSet; import org.briarproject.api.crypto.CryptoComponent; @@ -70,11 +69,6 @@ public class SymmetricIdenticon extends IdenticonView { protected boolean isCellVisible(int row, int column) { return getByte(3 + row * CENTER_COLUMN_INDEX + getSymmetricColumnIndex(column)) >= 0; } - - @Override - protected int getIconColor() { - return Color.rgb(getByte(0) + 128, getByte(1) + 128, getByte(2) + 128); - } }; }