From 9459f89a1d54f4c978df627fb5aaa45365f778e0 Mon Sep 17 00:00:00 2001 From: Christian Schneppe Date: Fri, 12 Apr 2019 21:33:02 +0200 Subject: use styled warning for message error --- .../java/de/pixart/messenger/ui/ConversationsOverviewFragment.java | 4 ++-- .../java/de/pixart/messenger/ui/PublishProfilePictureActivity.java | 6 +++--- src/main/java/de/pixart/messenger/ui/XmppActivity.java | 6 ------ src/main/java/de/pixart/messenger/ui/adapter/MessageAdapter.java | 6 +++++- 4 files changed, 10 insertions(+), 12 deletions(-) (limited to 'src/main/java/de/pixart/messenger') diff --git a/src/main/java/de/pixart/messenger/ui/ConversationsOverviewFragment.java b/src/main/java/de/pixart/messenger/ui/ConversationsOverviewFragment.java index 4ce263d0a..4f85c97c6 100644 --- a/src/main/java/de/pixart/messenger/ui/ConversationsOverviewFragment.java +++ b/src/main/java/de/pixart/messenger/ui/ConversationsOverviewFragment.java @@ -74,7 +74,7 @@ public class ConversationsOverviewFragment extends XmppFragment { public static Conversation getSuggestion(Activity activity) { final Conversation exception; Fragment fragment = activity.getFragmentManager().findFragmentById(R.id.main_fragment); - if (fragment != null && fragment instanceof ConversationsOverviewFragment) { + if (fragment instanceof ConversationsOverviewFragment) { exception = ((ConversationsOverviewFragment) fragment).swipedConversation.peek(); } else { exception = null; @@ -84,7 +84,7 @@ public class ConversationsOverviewFragment extends XmppFragment { public static Conversation getSuggestion(Activity activity, Conversation exception) { Fragment fragment = activity.getFragmentManager().findFragmentById(R.id.main_fragment); - if (fragment != null && fragment instanceof ConversationsOverviewFragment) { + if (fragment instanceof ConversationsOverviewFragment) { List conversations = ((ConversationsOverviewFragment) fragment).conversations; if (conversations.size() > 0) { Conversation suggestion = conversations.get(0); diff --git a/src/main/java/de/pixart/messenger/ui/PublishProfilePictureActivity.java b/src/main/java/de/pixart/messenger/ui/PublishProfilePictureActivity.java index e38591848..db81e573b 100644 --- a/src/main/java/de/pixart/messenger/ui/PublishProfilePictureActivity.java +++ b/src/main/java/de/pixart/messenger/ui/PublishProfilePictureActivity.java @@ -68,7 +68,7 @@ public class PublishProfilePictureActivity extends XmppActivity implements XmppC public void onAvatarPublicationFailed(int res) { runOnUiThread(() -> { hintOrWarning.setText(res); - hintOrWarning.setTextColor(getWarningTextColor()); + hintOrWarning.setTextAppearance(this, R.style.TextAppearance_Conversations_Body1_Warning); hintOrWarning.setVisibility(View.VISIBLE); publishing = false; togglePublishButton(true, R.string.publish); @@ -208,7 +208,7 @@ public class PublishProfilePictureActivity extends XmppActivity implements XmppC if (bm == null) { togglePublishButton(false, R.string.publish); this.hintOrWarning.setVisibility(View.VISIBLE); - this.hintOrWarning.setTextColor(getWarningTextColor()); + this.hintOrWarning.setTextAppearance(this, R.style.TextAppearance_Conversations_Body1_Warning); this.hintOrWarning.setText(R.string.error_publish_avatar_converting); return; } @@ -219,7 +219,7 @@ public class PublishProfilePictureActivity extends XmppActivity implements XmppC } else { togglePublishButton(false, R.string.publish); this.hintOrWarning.setVisibility(View.VISIBLE); - this.hintOrWarning.setTextColor(getWarningTextColor()); + this.hintOrWarning.setTextAppearance(this, R.style.TextAppearance_Conversations_Body1_Warning); if (account.getStatus() == Account.State.ONLINE) { this.hintOrWarning.setText(R.string.error_publish_avatar_no_server_support); } else { diff --git a/src/main/java/de/pixart/messenger/ui/XmppActivity.java b/src/main/java/de/pixart/messenger/ui/XmppActivity.java index e482f3879..ef4882f43 100644 --- a/src/main/java/de/pixart/messenger/ui/XmppActivity.java +++ b/src/main/java/de/pixart/messenger/ui/XmppActivity.java @@ -112,7 +112,6 @@ public abstract class XmppActivity extends ActionBarActivity { public XmppConnectionService xmppConnectionService; public boolean xmppConnectionServiceBound = false; - protected int mColorRed; protected int mColorWarningButton; protected int mColorWhite; @@ -405,7 +404,6 @@ public abstract class XmppActivity extends ActionBarActivity { ExceptionHelper.init(getApplicationContext()); new EmojiService(this).init(getPreferences().getBoolean(USE_BUNDLED_EMOJIS, getResources().getBoolean(R.bool.use_bundled_emoji))); this.isCameraFeatureAvailable = getPackageManager().hasSystemFeature(PackageManager.FEATURE_CAMERA); - mColorRed = ContextCompat.getColor(this, R.color.red800); if (isDarkTheme()) { mColorWarningButton = ContextCompat.getColor(this, R.color.warning_button_dark); } else { @@ -921,10 +919,6 @@ public abstract class XmppActivity extends ActionBarActivity { } } - public int getWarningTextColor() { - return this.mColorRed; - } - public int getWarningButtonColor() { return this.mColorWarningButton; } diff --git a/src/main/java/de/pixart/messenger/ui/adapter/MessageAdapter.java b/src/main/java/de/pixart/messenger/ui/adapter/MessageAdapter.java index a97f68cd8..b3fd88041 100644 --- a/src/main/java/de/pixart/messenger/ui/adapter/MessageAdapter.java +++ b/src/main/java/de/pixart/messenger/ui/adapter/MessageAdapter.java @@ -290,7 +290,11 @@ public class MessageAdapter extends ArrayAdapter implements CopyTextVie break; } if (error && type == SENT) { - viewHolder.time.setTextAppearance(getContext(), R.style.TextAppearance_Conversations_Caption_Warning); + if (darkBackground) { + viewHolder.time.setTextAppearance(getContext(), R.style.TextAppearance_Conversations_Caption_Warning_OnDark); + } else { + viewHolder.time.setTextAppearance(getContext(), R.style.TextAppearance_Conversations_Caption_Warning); + } DownloadableFile file = activity.xmppConnectionService.getFileBackend().getFile(message); if (file.exists()) { if (activity.xmppConnectionService.mHttpConnectionManager.getAutoAcceptFileSize() >= message.getFileParams().size) { -- cgit v1.2.3