ZwitscherA icon indicating copy to clipboard operation
ZwitscherA copied to clipboard

Read detection not good in v0.91

Open pilhuhn opened this issue 12 years ago • 1 comments

Often statuses that should be marked as read aren't. Those are local ones from the same or other timelines.

pilhuhn avatar Dec 30 '12 20:12 pilhuhn

This patch makes the read detection more aggressive, but also "colors" the ones we just browsed over. I am not yet sure if that is a good idea. And it probably gets confusing if the read items would be minimized on the fly and thus the ones one just looked at would "disappear"

===================================================================
--- src/de/bsd/zwitscher/StatusAdapter.java (revision 73986724814260132b091722579d11e4525a1631)
+++ src/de/bsd/zwitscher/StatusAdapter.java (revision )
@@ -23,7 +23,6 @@
 import twitter4j.URLEntity;
 import twitter4j.User;

-import java.net.URL;
 import java.util.HashSet;
 import java.util.List;
 import java.util.Set;
@@ -115,6 +114,9 @@
                 isOld = true;
             } else if (readIds.contains(oid)) {
                 convertView.setBackgroundColor(Color.rgb(0,0,40)); // todo debug color
+                isOld=true;
+            } else if (newOlds.contains(oid)) {
+                convertView.setBackgroundColor(Color.rgb(0,0,70));
                 isOld=true;
             } else if (status.isRetweet() && (readIds.contains(rtStatusId) || newOlds.contains(rtStatusId))) {
                 convertView.setBackgroundColor(Color.rgb(40,0,0));

pilhuhn avatar Feb 03 '13 21:02 pilhuhn