summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorJason A. Donenfeld <Jason@zx2c4.com>2018-07-29 16:52:17 +0200
committerJason A. Donenfeld <Jason@zx2c4.com>2018-07-29 17:21:30 +0200
commit219f4e80166800a9a44f84f9cc097fbed90e225e (patch)
treee0dd07d1adb3c632a44a677ca7b348a54e491010
parent6558140a7ca7eda2e39c75cbb587a6f25e3cda4e (diff)
MSF: make it more obvious what needs to be done
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
-rw-r--r--app/src/main/res/drawable/list_item_background.xml9
-rw-r--r--app/src/main/res/drawable/list_item_background_anim.xml4
2 files changed, 9 insertions, 4 deletions
diff --git a/app/src/main/res/drawable/list_item_background.xml b/app/src/main/res/drawable/list_item_background.xml
index 28d33737..f82df249 100644
--- a/app/src/main/res/drawable/list_item_background.xml
+++ b/app/src/main/res/drawable/list_item_background.xml
@@ -1,5 +1,10 @@
<?xml version="1.0" encoding="utf-8"?>
<selector xmlns:android="http://schemas.android.com/apk/res/android">
- <item android:drawable="@color/list_item_activated" android:state_activated="true" /> <!-- TODO(msf): themeify this -->
- <item android:drawable="@android:color/transparent" /> <!-- TODO(msf): themeify this -->
+ <!-- TODO(msf): somehow make an additional activated color entry here (or elsewhere?) that
+ is in use only in multiselect mode, and one that's in use only in ordinary
+ select mode. In otherwords, we don't want to use the same background color
+ for both multiselect and normal select (on two column machines).
+ -->
+ <item android:drawable="@color/list_item_activated" android:state_activated="true" /> <!-- TODO(msf): get color from theme with ?android/attr: -->
+ <item android:drawable="@android:color/transparent" /> <!-- TODO(msf): get color from theme with ?android/attr: -->
</selector>
diff --git a/app/src/main/res/drawable/list_item_background_anim.xml b/app/src/main/res/drawable/list_item_background_anim.xml
index 213130c7..8d9ffa42 100644
--- a/app/src/main/res/drawable/list_item_background_anim.xml
+++ b/app/src/main/res/drawable/list_item_background_anim.xml
@@ -1,6 +1,6 @@
<?xml version="1.0" encoding="utf-8"?>
<ripple xmlns:android="http://schemas.android.com/apk/res/android"
- android:color="@color/list_item_ripple"> <!-- TODO(msf): themeify this -->
+ android:color="@color/list_item_ripple"> <!-- TODO(msf): get color from theme with ?android/attr: -->
<item android:drawable="@drawable/list_item_background" />
- <item android:id="@android:id/mask" android:drawable="@android:color/white" />
+ <item android:id="@android:id/mask" android:drawable="@android:color/white" /> <!-- TODO(msf): get color from theme with ?android/attr: -->
</ripple>