#154 Horizontal lines (separators) are now above Day headers, not below them

Author: yvolk

app/src/main/res/layout/day_header_center.xml

@@ -11,18 +11,18 @@
               android:paddingBottom="@dimen/entry_bottom_padding">
 
     <TextView
-        android:id="@+id/day_header_title"
-        style="@style/DayHeaderTitle"
+        android:id="@+id/day_header_separator"
+        style="@style/DayHeaderSeparator"
         android:layout_width="fill_parent"
-        android:layout_height="fill_parent"
-        android:gravity="center"
+        android:layout_height="1dp"
         tools:ignore="SelectableText" />
 
     <TextView
-        android:id="@+id/day_header_separator"
-        style="@style/DayHeaderSeparator"
+        android:id="@+id/day_header_title"
+        style="@style/DayHeaderTitle"
         android:layout_width="fill_parent"
-        android:layout_height="1dp"
+        android:layout_height="fill_parent"
+        android:gravity="center"
         tools:ignore="SelectableText" />
 
 </LinearLayout>

app/src/main/res/layout/day_header_left.xml

@@ -11,18 +11,18 @@
               android:paddingBottom="@dimen/entry_bottom_padding">
 
     <TextView
-        android:id="@+id/day_header_title"
-        style="@style/DayHeaderTitle"
+        android:id="@+id/day_header_separator"
+        style="@style/DayHeaderSeparator"
         android:layout_width="fill_parent"
-        android:layout_height="fill_parent"
-        android:gravity="left"
+        android:layout_height="1dp"
         tools:ignore="SelectableText" />
 
     <TextView
-        android:id="@+id/day_header_separator"
-        style="@style/DayHeaderSeparator"
+        android:id="@+id/day_header_title"
+        style="@style/DayHeaderTitle"
         android:layout_width="fill_parent"
-        android:layout_height="1dp"
+        android:layout_height="fill_parent"
+        android:gravity="left"
         tools:ignore="SelectableText" />
 
 </LinearLayout>

app/src/main/res/layout/day_header_right.xml

@@ -11,18 +11,18 @@
               android:paddingBottom="@dimen/entry_bottom_padding">
 
     <TextView
-        android:id="@+id/day_header_title"
-        style="@style/DayHeaderTitle"
+        android:id="@+id/day_header_separator"
+        style="@style/DayHeaderSeparator"
         android:layout_width="fill_parent"
-        android:layout_height="fill_parent"
-        android:gravity="right"
+        android:layout_height="1dp"
         tools:ignore="SelectableText" />
 
     <TextView
-        android:id="@+id/day_header_separator"
-        style="@style/DayHeaderSeparator"
+        android:id="@+id/day_header_title"
+        style="@style/DayHeaderTitle"
         android:layout_width="fill_parent"
-        android:layout_height="1dp"
+        android:layout_height="fill_parent"
+        android:gravity="right"
         tools:ignore="SelectableText" />
 
 </LinearLayout>

app/src/main/res/values/dimens.xml

@@ -12,9 +12,9 @@
     <dimen name="event_time_width">80dp</dimen>
     <dimen name="entry_bottom_padding">2dp</dimen>
     <dimen name="day_header_padding_left">12dp</dimen>
-    <dimen name="day_header_padding_top">12dp</dimen>
+    <dimen name="day_header_padding_top">2dp</dimen>
     <dimen name="day_header_padding_right">10dp</dimen>
-    <dimen name="day_header_padding_bottom">2dp</dimen>
+    <dimen name="day_header_padding_bottom">12dp</dimen>
     <!-- Default screen margins, per the Android Design guidelines. -->
     <dimen name="activity_horizontal_margin">16dp</dimen>
     <dimen name="activity_vertical_margin">16dp</dimen>

app/src/main/res/values/styles.xml

@@ -14,7 +14,7 @@
 
     <style name="DayHeaderTitle" parent="android:Widget.Holo.TextView">
         <item name="android:ellipsize">marquee</item>
-        <item name="android:gravity">right|bottom</item>
+        <item name="android:gravity">right|top</item>
         <item name="android:paddingBottom">@dimen/day_header_padding_bottom</item>
         <item name="android:paddingRight">@dimen/day_header_padding_right</item>
         <item name="android:paddingTop">@dimen/day_header_padding_top</item>