Issue #57

IndiePass · Sep 8, 2018 · 9e5bed6 · 9e5bed6
1 parent 84408cd
commit 9e5bed6
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/app/src/main/assets/changelog b/app/src/main/assets/changelog
@@ -6,6 +6,7 @@ Icons from <a href="https://material.io/tools/icons/">Material Design</a> and <a
 <h2>Changelog</h2>
 <h3>10/09/2018 - 0.14</h3>
 <ul>
+    <li>Cleaner date formatting in timeline</li>
     <li>Check required fields on post types</li>
     <li>Cleaner positioning of 'new'</li>
     <li>Send post status (published or draft)</li>

diff --git a/app/src/main/java/com/indieweb/indigenous/microsub/timeline/TimelineListAdapter.java b/app/src/main/java/com/indieweb/indigenous/microsub/timeline/TimelineListAdapter.java
@@ -31,7 +31,6 @@
 import com.indieweb.indigenous.micropub.post.RsvpActivity;
 import com.indieweb.indigenous.model.TimelineItem;
 
-import java.text.DateFormat;
 import java.text.ParseException;
 import java.text.SimpleDateFormat;
 import java.util.Date;
@@ -137,14 +136,13 @@ public View getView(int position, View convertView, ViewGroup parent) {
             }
 
             // Published.
-            DateFormat df = new SimpleDateFormat("yyyy-MM-dd'T'kk:mm:ssZ");
+            SimpleDateFormat formatIn = new SimpleDateFormat("yyyy-MM-dd'T'kk:mm:ssZ");
+            SimpleDateFormat formatOut = new SimpleDateFormat("dd MMM yyyy kk:mm");
             Date result;
             try {
-                // TODO use format (cf event)
-                result = df.parse(item.getPublished());
+                result = formatIn.parse(item.getPublished());
                 holder.published.setVisibility(View.VISIBLE);
-                String published = result.toString();
-                holder.published.setText(published);
+                holder.published.setText(formatOut.format(result));
             } catch (ParseException ignored) {
                 holder.published.setVisibility(View.GONE);
             }
@@ -173,7 +171,6 @@ public View getView(int position, View convertView, ViewGroup parent) {
                 holder.name.setVisibility(View.GONE);
             }
 
-            // TODO we should probably create subtype classes per 'type'
             String ContextData = "";
             if (item.getType().equals("bookmark-of") || item.getType().equals("in-reply-to") || item.getType().equals("like-of") || item.getType().equals("checkin")) {
                 String ContextText = "";