diff --git a/briar-android/artwork/ic_exclamation_mark.svg b/briar-android/artwork/ic_exclamation_mark.svg new file mode 100644 index 000000000..204d21e72 --- /dev/null +++ b/briar-android/artwork/ic_exclamation_mark.svg @@ -0,0 +1,5 @@ + + + + + diff --git a/briar-android/src/main/java/org/briarproject/briar/android/mailbox/MailboxStatusFragment.java b/briar-android/src/main/java/org/briarproject/briar/android/mailbox/MailboxStatusFragment.java index 6c2584b05..dcaf17304 100644 --- a/briar-android/src/main/java/org/briarproject/briar/android/mailbox/MailboxStatusFragment.java +++ b/briar-android/src/main/java/org/briarproject/briar/android/mailbox/MailboxStatusFragment.java @@ -1,7 +1,6 @@ package org.briarproject.briar.android.mailbox; import android.content.Context; -import android.content.res.ColorStateList; import android.os.Bundle; import android.os.Handler; import android.os.Looper; @@ -21,7 +20,6 @@ import org.briarproject.nullsafety.ParametersNotNullByDefault; import javax.inject.Inject; -import androidx.annotation.ColorRes; import androidx.annotation.DrawableRes; import androidx.annotation.Nullable; import androidx.annotation.UiThread; @@ -33,8 +31,6 @@ import androidx.lifecycle.ViewModelProvider; import static android.view.View.GONE; import static android.view.View.INVISIBLE; import static android.view.View.VISIBLE; -import static androidx.core.content.ContextCompat.getColor; -import static androidx.core.widget.ImageViewCompat.setImageTintList; import static androidx.transition.TransitionManager.beginDelayedTransition; import static org.briarproject.bramble.api.mailbox.MailboxConstants.API_CLIENT_TOO_OLD; import static org.briarproject.briar.android.AppModule.getAndroidComponent; @@ -141,24 +137,20 @@ public class MailboxStatusFragment extends Fragment { } private void onMailboxStateChanged(MailboxStatus status) { - @ColorRes int tintRes; @DrawableRes int iconRes; String title; String message = null; if (status.hasProblem(System.currentTimeMillis())) { - tintRes = R.color.briar_red_500; title = getString(R.string.mailbox_status_failure_title); iconRes = R.drawable.ic_error; showUnlinkWarning = false; wizardButton.setVisibility(VISIBLE); } else if (status.getAttemptsSinceSuccess() > 0) { - iconRes = R.drawable.ic_help_outline_white; + iconRes = R.drawable.ic_exclamation_mark; title = getString(R.string.mailbox_status_problem_title); - tintRes = R.color.briar_orange_500; showUnlinkWarning = false; wizardButton.setVisibility(VISIBLE); } else if (status.getMailboxCompatibility() < 0) { - tintRes = R.color.briar_red_500; if (status.getMailboxCompatibility() == API_CLIENT_TOO_OLD) { title = getString(R.string.mailbox_status_app_too_old_title); message = @@ -175,13 +167,10 @@ public class MailboxStatusFragment extends Fragment { } else { iconRes = R.drawable.ic_check_circle; title = getString(R.string.mailbox_status_connected_title); - tintRes = R.color.briar_brand_green; showUnlinkWarning = true; wizardButton.setVisibility(GONE); } imageView.setImageResource(iconRes); - int color = getColor(requireContext(), tintRes); - setImageTintList(imageView, ColorStateList.valueOf(color)); statusTitleView.setText(title); if (message == null) { statusMessageView.setVisibility(GONE); diff --git a/briar-android/src/main/res/drawable/ic_exclamation_mark.xml b/briar-android/src/main/res/drawable/ic_exclamation_mark.xml new file mode 100644 index 000000000..7e728cb86 --- /dev/null +++ b/briar-android/src/main/res/drawable/ic_exclamation_mark.xml @@ -0,0 +1,16 @@ + + + + + diff --git a/briar-android/src/main/res/values-night/color.xml b/briar-android/src/main/res/values-night/color.xml index f74158736..5cffcc605 100644 --- a/briar-android/src/main/res/values-night/color.xml +++ b/briar-android/src/main/res/values-night/color.xml @@ -30,5 +30,6 @@ #64748B #334155 @color/briar_lime_400_new + @color/briar_orange_400 @color/briar_red_500_new diff --git a/briar-android/src/main/res/values/color.xml b/briar-android/src/main/res/values/color.xml index ced779896..973a0a0fc 100644 --- a/briar-android/src/main/res/values/color.xml +++ b/briar-android/src/main/res/values/color.xml @@ -10,7 +10,9 @@ #418cd8 #fed69f + #FBBF24 #fc9403 + #F59E0B #db3b21 #EF4444 @@ -60,6 +62,7 @@ #9CA3AF #E5E7EB @color/briar_lime_600_new + @color/briar_orange_500_new @color/briar_red_600