From 0f764664d4a5e7777bd9d02ba22fe1de25ff78b9 Mon Sep 17 00:00:00 2001
From: Andy2 <andy2@trand>
Date: Thu, 10 Feb 2011 06:11:54 -0800
Subject: [PATCH] add comment about failed attempt to deliniate list items by
 having their backgrounds alternate slightly.

---
 .../src/org/eehouse/android/xw4/GameListAdapter.java | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/xwords4/android/XWords4/src/org/eehouse/android/xw4/GameListAdapter.java b/xwords4/android/XWords4/src/org/eehouse/android/xw4/GameListAdapter.java
index d22507358..abf1e927b 100644
--- a/xwords4/android/XWords4/src/org/eehouse/android/xw4/GameListAdapter.java
+++ b/xwords4/android/XWords4/src/org/eehouse/android/xw4/GameListAdapter.java
@@ -104,6 +104,18 @@ public class GameListAdapter extends XWListAdapter {
                                   View.VISIBLE : View.GONE );
             m_viewsCache.put( path, layout );
         }
+
+        // this doesn't work.  Rather, it breaks highlighting because
+        // the background, if we don't set it, is a more complicated
+        // object like @android:drawable/list_selector_background.  I
+        // tried calling getBackground(), expecting to get a Drawable
+        // I could then clone and modify, but null comes back.  So
+        // layout must be inheriting its background from elsewhere or
+        // it gets set later, during layout.
+        // if ( (position%2) == 0 ) {
+        //     layout.setBackgroundColor( 0xFF3F3F3F );
+        // }
+
         return layout;
     } // getItem