diff --git a/app/src/main/java/com/nextcloud/talk/adapters/items/RoomItem.java b/app/src/main/java/com/nextcloud/talk/adapters/items/RoomItem.java index 4c5c8cfc6..de5681ee4 100644 --- a/app/src/main/java/com/nextcloud/talk/adapters/items/RoomItem.java +++ b/app/src/main/java/com/nextcloud/talk/adapters/items/RoomItem.java @@ -33,6 +33,7 @@ import com.nextcloud.talk.api.helpers.api.ApiHelper; import com.nextcloud.talk.api.models.json.rooms.Room; import com.nextcloud.talk.application.NextcloudTalkApplication; import com.nextcloud.talk.persistence.entities.UserEntity; +import com.nextcloud.talk.utils.ColorUtils; import com.nextcloud.talk.utils.glide.GlideApp; import java.util.List; @@ -113,6 +114,9 @@ public class RoomItem extends AbstractFlexibleItem holder.avatarImageView.setVisibility(View.VISIBLE); if (!TextUtils.isEmpty(room.getName())) { + holder.avatarImageView.setTextAndColorSeed(String.valueOf(room.getName(). + toUpperCase().charAt(0)), ColorUtils.colorSeed); + GlideUrl glideUrl = new GlideUrl(ApiHelper.getUrlForAvatarWithName(userEntity.getBaseUrl(), room.getName()), new LazyHeaders.Builder() .setHeader("Accept", "image/*") @@ -124,13 +128,12 @@ public class RoomItem extends AbstractFlexibleItem .skipMemoryCache(true) .diskCacheStrategy(DiskCacheStrategy.NONE) .load(glideUrl) - .placeholder(holder.avatarImageViewInvisible.getDrawable()) .circleCrop() .centerInside() .into(holder.avatarImageView); } else { - holder.avatarImageView.setDrawable(holder.avatarImageViewInvisible.getDrawable()); + holder.avatarImageView.setVisibility(View.GONE); } break; case ROOM_GROUP_CALL: @@ -162,8 +165,6 @@ public class RoomItem extends AbstractFlexibleItem public TextView roomLastPing; @BindView(R.id.avatar_image) public AvatarImageView avatarImageView; - @BindView(R.id.avatar_image_invisible) - public AvatarImageView avatarImageViewInvisible; /** * Default constructor. diff --git a/app/src/main/java/com/nextcloud/talk/adapters/items/UserItem.java b/app/src/main/java/com/nextcloud/talk/adapters/items/UserItem.java index 1eb3ca72f..ffedd3b6b 100644 --- a/app/src/main/java/com/nextcloud/talk/adapters/items/UserItem.java +++ b/app/src/main/java/com/nextcloud/talk/adapters/items/UserItem.java @@ -109,7 +109,6 @@ public class UserItem extends AbstractFlexibleItem .asBitmap() .skipMemoryCache(true) .diskCacheStrategy(DiskCacheStrategy.NONE) - .placeholder(holder.avatarImageViewInvisible.getDrawable()) .load(glideUrl) .circleCrop() .centerInside() @@ -128,8 +127,6 @@ public class UserItem extends AbstractFlexibleItem public TextView contactDisplayName; @BindView(R.id.avatar_image) public AvatarImageView avatarImageView; - @BindView(R.id.avatar_image_invisible) - public AvatarImageView avatarImageViewInvisible; /** * Default constructor. diff --git a/app/src/main/java/com/nextcloud/talk/controllers/SettingsController.java b/app/src/main/java/com/nextcloud/talk/controllers/SettingsController.java index 005c5071e..51e0c51c4 100644 --- a/app/src/main/java/com/nextcloud/talk/controllers/SettingsController.java +++ b/app/src/main/java/com/nextcloud/talk/controllers/SettingsController.java @@ -90,9 +90,6 @@ public class SettingsController extends BaseController { @BindView(R.id.avatar_image) AvatarImageView avatarImageView; - @BindView(R.id.avatar_image_invisible) - AvatarImageView avatarImageViewInvisible; - @BindView(R.id.display_name_text) TextView displayName; @@ -191,7 +188,7 @@ public class SettingsController extends BaseController { UserEntity userEntity = userUtils.getCurrentUser(); if (userEntity != null) { // Awful hack - avatarImageViewInvisible.setTextAndColorSeed(String.valueOf(userEntity.getDisplayName(). + avatarImageView.setTextAndColorSeed(String.valueOf(userEntity.getDisplayName(). toUpperCase().charAt(0)), ColorUtils.colorSeed); GlideUrl glideUrl = new GlideUrl(ApiHelper.getUrlForAvatarWithName(userEntity.getBaseUrl(), @@ -204,8 +201,7 @@ public class SettingsController extends BaseController { .load(glideUrl) .circleCrop() .centerInside() - .into(avatarImageView) - .onLoadFailed(avatarImageViewInvisible.getDrawable()); + .into(avatarImageView); displayName.setText(userEntity.getDisplayName()); } diff --git a/app/src/main/res/layout/controller_settings.xml b/app/src/main/res/layout/controller_settings.xml index 896eb5958..694dbfdce 100644 --- a/app/src/main/res/layout/controller_settings.xml +++ b/app/src/main/res/layout/controller_settings.xml @@ -34,17 +34,6 @@ android:layout_width="match_parent" android:layout_height="wrap_content"> - - - - - -