[Android] UI : Sidebar design cosmetics

[Alexandre Perraud]

vlc-ports/android | branch: master | Alexandre Perraud <4leyx4ndre at gmail.com> | Wed Jan 28 17:13:52 2015 +0100| [330d1d1626486d70e8a45787c2451e5e8ef6f807] | committer: Alexandre Perraud

UI : Sidebar design cosmetics

> http://git.videolan.org/gitweb.cgi/vlc-ports/android.git/?a=commit;h=330d1d1626486d70e8a45787c2451e5e8ef6f807
---

 vlc-android/res/layout/sidebar.xml      |   39 ++++++++++++-------------------
 vlc-android/res/layout/sidebar_item.xml |    2 +-
 2 files changed, 16 insertions(+), 25 deletions(-)

diff --git a/vlc-android/res/layout/sidebar.xml b/vlc-android/res/layout/sidebar.xml
index f2dc061..ad45286 100644
--- a/vlc-android/res/layout/sidebar.xml
+++ b/vlc-android/res/layout/sidebar.xml
@@ -10,11 +10,9 @@
             android:id="@+id/sidelist"
             android:layout_width="208dp"
             android:layout_height="match_parent"
-            android:divider="?attr/background_menu_divider"
-            android:dividerHeight="0.2dp"
             android:paddingTop="?attr/actionBarSize"
             android:cacheColorHint="#0000"
-            android:layout_above="@+id/sidebar_separator_about"
+            android:layout_above="@+id/sidebar_separator_settings"
             android:nextFocusForward="@id/sidelist"
             android:nextFocusUp="@id/sidelist"
             android:nextFocusDown="@id/sidelist"
@@ -23,48 +21,41 @@
         </ListView>
 
         <View
-            android:id="@+id/sidebar_separator_about"
+            android:id="@+id/sidebar_separator_settings"
             android:layout_width="match_parent"
             android:layout_height="1dp"
-            android:layout_above="@+id/about"
+            android:layout_above="@+id/settings"
             android:background="?attr/playlist_item_footer" />
 
         <ImageView
-            android:id="@+id/about_icon"
+            android:id="@+id/settings_icon"
             android:layout_width="32dp"
             android:layout_height="32dp"
             android:layout_alignParentLeft="true"
-            android:layout_above="@+id/sidebar_separator_settings"
+            android:layout_above="@+id/about_icon"
             android:layout_margin="10dp"
             android:clickable="true"
             android:onClick="onClick"
-            android:src="?attr/ic_menu_cone"/>
+            android:src="?attr/ic_menu_preferences"/>
 
         <TextView
-            android:id="@+id/about"
+            android:id="@+id/settings"
             android:layout_width="match_parent"
             android:layout_height="32dp"
-            android:layout_toRightOf="@+id/about_icon"
-            android:layout_above="@+id/sidebar_separator_settings"
+            android:layout_toRightOf="@+id/settings_icon"
+            android:layout_above="@+id/about_icon"
             android:gravity="left|center_vertical"
             android:layout_marginTop="10dp"
             android:layout_marginBottom="10dp"
             android:clickable="true"
             android:onClick="onClick"
-            android:text="@string/about"
+            android:text="@string/preferences"
             android:textColor="?attr/list_menu"
             android:textSize="16sp"
             android:fontFamily="sans-serif"/>
 
-        <View
-            android:id="@+id/sidebar_separator_settings"
-            android:layout_width="match_parent"
-            android:layout_height="1dp"
-            android:layout_above="@+id/settings"
-            android:background="?attr/playlist_item_footer" />
-
         <ImageView
-            android:id="@+id/settings_icon"
+            android:id="@+id/about_icon"
             android:layout_width="32dp"
             android:layout_height="32dp"
             android:layout_alignParentLeft="true"
@@ -72,20 +63,20 @@
             android:layout_margin="10dp"
             android:clickable="true"
             android:onClick="onClick"
-            android:src="?attr/ic_menu_preferences"/>
+            android:src="?attr/ic_menu_cone"/>
 
         <TextView
-            android:id="@+id/settings"
+            android:id="@+id/about"
             android:layout_width="match_parent"
             android:layout_height="32dp"
+            android:layout_toRightOf="@+id/about_icon"
             android:layout_alignParentBottom="true"
-            android:layout_toRightOf="@+id/settings_icon"
             android:gravity="left|center_vertical"
             android:layout_marginTop="10dp"
             android:layout_marginBottom="10dp"
             android:clickable="true"
             android:onClick="onClick"
-            android:text="@string/preferences"
+            android:text="@string/about"
             android:textColor="?attr/list_menu"
             android:textSize="16sp"
             android:fontFamily="sans-serif"/>
diff --git a/vlc-android/res/layout/sidebar_item.xml b/vlc-android/res/layout/sidebar_item.xml
index 3e515dc..1b304e0 100644
--- a/vlc-android/res/layout/sidebar_item.xml
+++ b/vlc-android/res/layout/sidebar_item.xml
@@ -9,7 +9,7 @@
     android:drawableLeft="@drawable/icon"
     android:drawablePadding="5dip"
     android:gravity="left|center_vertical"
-    android:padding="13dip"
+    android:padding="10dip"
     android:text="@string/title"
     android:textColor="?attr/list_menu"
     android:textSize="16sp" />