Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1475)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/android/java/res/values/dimens.xml

Issue 2844373003: [Home] Reduce bottom control container height (Closed)
Patch Set: Fix app menu positioning Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenu.java » ('j') | chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenu.java » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/java/res/values/dimens.xml
diff --git a/chrome/android/java/res/values/dimens.xml b/chrome/android/java/res/values/dimens.xml
index 51eb9bc09d205fb0b250bda57fe60e1a30480670..9aafdc5731bda47f67e6f62ee077bc18d346fc6d 100644
--- a/chrome/android/java/res/values/dimens.xml
+++ b/chrome/android/java/res/values/dimens.xml
@@ -204,8 +204,9 @@
<!-- Full Screen Dimensions -->
<!-- Should match toolbar_height_no_shadow -->
<dimen name="control_container_height">56dp</dimen>
- <dimen name="bottom_control_container_height">64dp</dimen>
- <dimen name="bottom_toolbar_top_margin">8dp</dimen>
+ <dimen name="bottom_control_container_height">56dp</dimen>
+ <dimen name="bottom_toolbar_top_margin">2dp</dimen>
+ <dimen name="bottom_toolbar_url_bar_top_margin">4dp</dimen>
<dimen name="custom_tabs_control_container_height">56dp</dimen>
<dimen name="webapp_control_container_height">22dp</dimen>
@@ -248,10 +249,10 @@
<dimen name="toolbar_shadow_height">8dp</dimen>
<dimen name="toolbar_progress_bar_height">2dp</dimen>
<dimen name="toolbar_button_width">48dp</dimen>
- <dimen name="toolbar_handle_height">4dp</dimen>
- <dimen name="toolbar_handle_width">40dp</dimen>
- <dimen name="toolbar_handle_corner_radius">2dp</dimen>
- <dimen name="toolbar_handle_margin_top">14dp</dimen>
+ <dimen name="toolbar_handle_height">3dp</dimen>
+ <dimen name="toolbar_handle_width">24dp</dimen>
+ <dimen name="toolbar_handle_corner_radius">1.5dp</dimen>
+ <dimen name="toolbar_handle_margin_top">12dp</dimen>
<dimen name="toolbar_edge_padding">8dp</dimen>
<dimen name="location_bar_google_g_width">24dp</dimen>
@@ -259,6 +260,7 @@
<!-- location_bar_google_g_width + 2 * location_bar_google_g_margin -->
<dimen name="location_bar_google_g_container_width">40dp</dimen>
<dimen name="location_bar_vertical_margin">8dp</dimen>
+ <dimen name="bottom_location_bar_vertical_margin">9dp</dimen>
<dimen name="location_bar_url_text_size">16sp</dimen>
<dimen name="location_bar_incognito_badge_padding">7dp</dimen>
<dimen name="location_bar_icon_width">32dp</dimen>
« no previous file with comments | « no previous file | chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenu.java » ('j') | chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenu.java » ('J')

Powered by Google App Engine
This is Rietveld 408576698