[Download Home] Adjust space display again

chrome/android/java/res/layout/download_manager_ui_space_widget.xml

 <?xml version="1.0" encoding="utf-8"?>
 <!-- Copyright 2016 The Chromium Authors. All rights reserved.
      Use of this source code is governed by a BSD-style license that can be
      found in the LICENSE file. -->
 
 <merge xmlns:android="http://schemas.android.com/apk/res/android">
 
+    <!-- Values in this file are a best effort attempt to match the dimensions in the "spec", which
+         had to be heavily interpreted for implementation.
+
+         https://bugs.chromium.org/p/chromium/issues/detail?id=658246&desc=2#c18
+    -->

[Theresa, 2016/11/29 00:37:13]

While this is certainly useful for reviewing the CL, I'm not sure the verbosity
is necessary when looking back at the code later. Maybe it should read:

<!-- Widget style adapted from the UX spec in crbug.com/658246 -->

[gone, 2016/11/29 00:48:20]

Was trying to refer to the comments following specific comment with the images;
went halfway and kept the link directly pointing at the images UX provided.

     <LinearLayout
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
             android:layout_marginStart="@dimen/drawer_margin_large"
             android:layout_marginEnd="@dimen/drawer_margin_large"
+            android:paddingTop="26dp"
+            android:paddingBottom="24dp"
             android:orientation="vertical" >
 
         <TextView
                 android:id="@+id/size_downloaded"
                 android:layout_width="match_parent"
                 android:layout_height="wrap_content"
                 android:singleLine="true"
                 android:textColor="@color/light_active_color"
                 android:textSize="20sp" />
 
-        <!-- The widget ends up claiming 32dp of space, vertically. -->
+        <!-- The progress bar uses 32dp of space, vertically, including spacing. -->
         <ProgressBar
                 android:id="@+id/space_bar"
                 style="@style/MaterialProgressBar"
                 android:layout_width="match_parent"
                 android:layout_height="4dp"
                 android:layout_marginTop="14dp"
                 android:layout_marginBottom="14dp" />
 
         <TextView
                 android:id="@+id/size_other_apps"
                 android:layout_width="match_parent"
                 android:layout_height="wrap_content"
                 android:singleLine="true"
                 android:textColor="@color/google_grey_600"
-                android:textSize="14sp" />
+                android:textSize="12sp" />
 
         <TextView
                 android:id="@+id/size_free"
                 android:layout_width="match_parent"
                 android:layout_height="wrap_content"
-                android:layout_marginBottom="22dp"
                 android:singleLine="true"
-                android:textColor="@color/material_progressbar_bg"
-                android:textSize="14sp" />
+                android:textColor="@color/google_grey_400"
+                android:textSize="12sp" />
 
     </LinearLayout>
 
 </merge>