diff options
author | Stefan Niedermann <info@niedermann.it> | 2024-01-16 11:21:57 +0300 |
---|---|---|
committer | Stefan Niedermann <info@niedermann.it> | 2024-01-16 11:21:57 +0300 |
commit | 47145c1c06b2de007dc67b0182235915500d4bb9 (patch) | |
tree | 8927f0779a75df8ecc557dfad0cb6e86b6fda90b /app/src/main/java/it/niedermann/nextcloud/deck/ui/card | |
parent | b66c01dcf779bc57edb161da036989c40e765d07 (diff) |
chore(done): Use filled checkbox item for tasks
Signed-off-by: Stefan Niedermann <info@niedermann.it>
Diffstat (limited to 'app/src/main/java/it/niedermann/nextcloud/deck/ui/card')
4 files changed, 20 insertions, 6 deletions
diff --git a/app/src/main/java/it/niedermann/nextcloud/deck/ui/card/AbstractCardViewHolder.java b/app/src/main/java/it/niedermann/nextcloud/deck/ui/card/AbstractCardViewHolder.java index 7653f4ff3..193fc7a2c 100644 --- a/app/src/main/java/it/niedermann/nextcloud/deck/ui/card/AbstractCardViewHolder.java +++ b/app/src/main/java/it/niedermann/nextcloud/deck/ui/card/AbstractCardViewHolder.java @@ -58,6 +58,8 @@ public abstract class AbstractCardViewHolder extends RecyclerView.ViewHolder { if (utils != null) { utils.platform.colorImageView(getNotSyncedYet(), ColorRole.PRIMARY); + utils.platform.colorImageView(getCardMenu(), ColorRole.ON_SURFACE); + utils.platform.colorTextView(getCardTitle(), ColorRole.ON_SURFACE); } // TODO should be discussed with UX // utils.material.themeCardView(getCard()); @@ -95,7 +97,7 @@ public abstract class AbstractCardViewHolder extends RecyclerView.ViewHolder { protected abstract TextView getCardTitle(); - protected abstract View getCardMenu(); + protected abstract ImageView getCardMenu(); protected abstract MaterialCardView getCard(); diff --git a/app/src/main/java/it/niedermann/nextcloud/deck/ui/card/CompactCardViewHolder.java b/app/src/main/java/it/niedermann/nextcloud/deck/ui/card/CompactCardViewHolder.java index 7c53451a5..8dc4a3e48 100644 --- a/app/src/main/java/it/niedermann/nextcloud/deck/ui/card/CompactCardViewHolder.java +++ b/app/src/main/java/it/niedermann/nextcloud/deck/ui/card/CompactCardViewHolder.java @@ -79,7 +79,7 @@ public class CompactCardViewHolder extends AbstractCardViewHolder { } @Override - protected View getCardMenu() { + protected ImageView getCardMenu() { return binding.cardMenu; } diff --git a/app/src/main/java/it/niedermann/nextcloud/deck/ui/card/DefaultCardOnlyTitleViewHolder.java b/app/src/main/java/it/niedermann/nextcloud/deck/ui/card/DefaultCardOnlyTitleViewHolder.java index 1b6b2b3a0..4e16f8127 100644 --- a/app/src/main/java/it/niedermann/nextcloud/deck/ui/card/DefaultCardOnlyTitleViewHolder.java +++ b/app/src/main/java/it/niedermann/nextcloud/deck/ui/card/DefaultCardOnlyTitleViewHolder.java @@ -1,6 +1,5 @@ package it.niedermann.nextcloud.deck.ui.card; -import android.view.View; import android.view.View.OnClickListener; import android.view.View.OnLongClickListener; import android.widget.ImageView; @@ -51,7 +50,7 @@ public class DefaultCardOnlyTitleViewHolder extends AbstractCardViewHolder { } @Override - protected View getCardMenu() { + protected ImageView getCardMenu() { return binding.cardMenu; } diff --git a/app/src/main/java/it/niedermann/nextcloud/deck/ui/card/DefaultCardViewHolder.java b/app/src/main/java/it/niedermann/nextcloud/deck/ui/card/DefaultCardViewHolder.java index 083480736..20ce6dd1d 100644 --- a/app/src/main/java/it/niedermann/nextcloud/deck/ui/card/DefaultCardViewHolder.java +++ b/app/src/main/java/it/niedermann/nextcloud/deck/ui/card/DefaultCardViewHolder.java @@ -13,6 +13,9 @@ import androidx.annotation.Nullable; import androidx.core.content.ContextCompat; import com.google.android.material.card.MaterialCardView; +import com.nextcloud.android.common.ui.theme.utils.ColorRole; + +import java.util.stream.Stream; import it.niedermann.nextcloud.deck.R; import it.niedermann.nextcloud.deck.databinding.ItemCardDefaultBinding; @@ -79,7 +82,7 @@ public class DefaultCardViewHolder extends AbstractCardViewHolder { final var taskStatus = fullCard.getCard().getTaskStatus(); if (taskStatus.taskCount > 0) { binding.cardCountTasks.setText(context.getResources().getString(R.string.task_count, String.valueOf(taskStatus.doneCount), String.valueOf(taskStatus.taskCount))); - binding.cardCountTasks.setCompoundDrawablesWithIntrinsicBounds(ContextCompat.getDrawable(context, R.drawable.ic_check_grey600_24dp), null, null, null); + binding.cardCountTasks.setCompoundDrawablesWithIntrinsicBounds(ContextCompat.getDrawable(context, R.drawable.ic_check_box_24), null, null, null); binding.cardCountTasks.setVisibility(View.VISIBLE); } else { final String description = fullCard.getCard().getDescription(); @@ -91,6 +94,16 @@ public class DefaultCardViewHolder extends AbstractCardViewHolder { binding.cardCountTasks.setVisibility(View.GONE); } } + + if (utils != null) { + Stream.of( + binding.cardCountAttachments, + binding.cardCountTasks, + binding.cardCountComments + ).forEach(v -> { + utils.platform.colorTextView(v, ColorRole.ON_SURFACE_VARIANT ); + }); + } } @Override @@ -109,7 +122,7 @@ public class DefaultCardViewHolder extends AbstractCardViewHolder { } @Override - protected View getCardMenu() { + protected ImageView getCardMenu() { return binding.cardMenu; } |