From 41b68125d39c7d101f91459a5da79cf647619500 Mon Sep 17 00:00:00 2001 From: Nick Bradbury Date: Thu, 4 Dec 2025 16:13:47 -0500 Subject: [PATCH 1/2] Updated bookmark icon to be more visible in dark mode --- .../wordpress/android/ui/reader/ReaderPostListFragment.kt | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/WordPress/src/main/java/org/wordpress/android/ui/reader/ReaderPostListFragment.kt b/WordPress/src/main/java/org/wordpress/android/ui/reader/ReaderPostListFragment.kt index 20b9e9d3cb9a..ea475f22a1a9 100644 --- a/WordPress/src/main/java/org/wordpress/android/ui/reader/ReaderPostListFragment.kt +++ b/WordPress/src/main/java/org/wordpress/android/ui/reader/ReaderPostListFragment.kt @@ -31,6 +31,7 @@ import androidx.core.view.isVisible import androidx.lifecycle.ViewModelProvider import androidx.lifecycle.lifecycleScope import androidx.recyclerview.widget.RecyclerView +import com.google.android.material.R as MaterialR import com.google.android.material.dialog.MaterialAlertDialogBuilder import com.google.android.material.snackbar.Snackbar import com.google.android.material.tabs.TabLayout @@ -143,6 +144,7 @@ import org.wordpress.android.util.StringUtils import org.wordpress.android.util.ToastUtils import org.wordpress.android.util.WPActivityUtils import org.wordpress.android.util.config.SeenUnseenWithCounterFeatureConfig +import org.wordpress.android.util.extensions.getColorFromAttribute import org.wordpress.android.util.image.ImageManager import org.wordpress.android.viewmodel.Event import org.wordpress.android.widgets.AppReviewManager.incrementInteractions @@ -1849,6 +1851,11 @@ class ReaderPostListFragment : ViewPagerFragment(), OnPostSelectedListener, OnFo requireContext(), R.drawable.ic_bookmark_grey_dark_18dp ) + + val tint = requireContext().getColorFromAttribute(MaterialR.attr.colorOnBackground) + drawable?.setTint(tint) + drawable?.alpha = 150 + drawable!!.setBounds( 0, 0, From 1af6db26b76b62f49e4a8f23e9faee7193f5e1f2 Mon Sep 17 00:00:00 2001 From: Nick Bradbury Date: Thu, 4 Dec 2025 16:21:43 -0500 Subject: [PATCH 2/2] Use a const for the alpha --- .../org/wordpress/android/ui/reader/ReaderPostListFragment.kt | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/WordPress/src/main/java/org/wordpress/android/ui/reader/ReaderPostListFragment.kt b/WordPress/src/main/java/org/wordpress/android/ui/reader/ReaderPostListFragment.kt index ea475f22a1a9..f8dd8c9002c8 100644 --- a/WordPress/src/main/java/org/wordpress/android/ui/reader/ReaderPostListFragment.kt +++ b/WordPress/src/main/java/org/wordpress/android/ui/reader/ReaderPostListFragment.kt @@ -1854,7 +1854,7 @@ class ReaderPostListFragment : ViewPagerFragment(), OnPostSelectedListener, OnFo val tint = requireContext().getColorFromAttribute(MaterialR.attr.colorOnBackground) drawable?.setTint(tint) - drawable?.alpha = 150 + drawable?.alpha = BOOKMARK_IMAGE_ALPHA drawable!!.setBounds( 0, @@ -2909,6 +2909,7 @@ class ReaderPostListFragment : ViewPagerFragment(), OnPostSelectedListener, OnFo private const val NO_POSITION = -1 private const val RECYCLER_DELAY_MS = 1000L private const val BOOKMARK_IMAGE_MULTIPLIER = 1.2 + private const val BOOKMARK_IMAGE_ALPHA = 150 private var hasPurgedReaderDb = false /*