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

Unified Diff: chrome/android/java/res/layout/location_bar.xml

Issue 1505913003: Add update menu item and app menu icon badge (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add code create 1dp transparent border in LocatoinBarPhone Created 5 years 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
Index: chrome/android/java/res/layout/location_bar.xml
diff --git a/chrome/android/java/res/layout/location_bar.xml b/chrome/android/java/res/layout/location_bar.xml
index 073253f97b6b8a1876dbd3ca1882bb8a40e94659..0dacd59cfd3bdab3802c928a1c33f73e4516acf9 100644
--- a/chrome/android/java/res/layout/location_bar.xml
+++ b/chrome/android/java/res/layout/location_bar.xml
@@ -50,14 +50,29 @@
android:layout_height="@dimen/toolbar_height_no_shadow"
android:layout_gravity="end|center_vertical"
android:visibility="gone">
- <org.chromium.chrome.browser.widget.TintedImageButton
- android:id="@+id/document_menu_button"
- style="@style/LocationBarButton"
- android:layout_height="match_parent"
- android:layout_width="match_parent"
- android:layoutDirection="locale"
- android:src="@drawable/btn_menu"
- android:contentDescription="@string/accessibility_toolbar_btn_menu" />
+ <FrameLayout
+ android:id="@+id/document_menu_btn_wrapper"
+ android:layout_width="fill_parent"
+ android:layout_height="fill_parent" >
+ <org.chromium.chrome.browser.widget.TintedImageButton
+ android:id="@+id/document_menu_button"
+ style="@style/LocationBarButton"
+ android:layout_height="match_parent"
+ android:layout_width="match_parent"
+ android:layoutDirection="locale"
+ android:src="@drawable/btn_menu"
+ android:contentDescription="@string/accessibility_toolbar_btn_menu" />
+ <ImageView
+ android:id="@+id/document_menu_badge"
+ android:src="@drawable/badge_update"
+ android:layout_height="@dimen/menu_badge_size"
+ android:layout_width="@dimen/menu_badge_size"
+ android:layout_marginBottom="13dp"
+ android:layout_marginEnd="6dp"
+ android:layout_gravity="end|bottom"
+ android:contentDescription="@string/accessibility_toolbar_update_badge"
+ android:visibility="gone" />
+ </FrameLayout>
<org.chromium.chrome.browser.widget.TintedImageButton
android:id="@+id/delete_button"
style="@style/LocationBarButton"

Powered by Google App Engine
This is Rietveld 408576698