New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 707094 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Move overflow menu icon row to the bottom of menu

Project Member Reported by k...@chromium.org, Mar 31 2017

Issue description

In order to make the toolbar actions more accessible ergonomically, we should move the overflow menu toolbar actions (e.g. refresh, bookmark, download) as a fixed pane to the bottom.
 

Comment 1 by k...@chromium.org, Mar 31 2017

Cc: k...@chromium.org
 Issue 707024  has been merged into this issue.
Summary: Move overflow menu icon row to the bottom of menu (was: Move overflow menu toolbar to the bottom)
Owner: twelling...@chromium.org
Status: Started (was: Available)
Screenshots from in-flight patch https://codereview.chromium.org/2821003003
Screenshot_20170417-232650.png
538 KB View Download
Screenshot_20170417-232705.png
651 KB View Download
Project Member

Comment 5 by bugdroid1@chromium.org, Apr 19 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7601678d95b575aab8d337d26559fc321409ee17

commit 7601678d95b575aab8d337d26559fc321409ee17
Author: twellington <twellington@chromium.org>
Date: Wed Apr 19 00:48:02 2017

[Home] Show the app menu icons as a footer

Move the icon row at the top of the app menu to a fixed footer when
Chrome Home is enabled.

BUG= 707094 

Review-Url: https://codereview.chromium.org/2821223004
Cr-Commit-Position: refs/heads/master@{#465448}

[add] https://crrev.com/7601678d95b575aab8d337d26559fc321409ee17/chrome/android/java/res/color/app_menu_button_tint.xml
[add] https://crrev.com/7601678d95b575aab8d337d26559fc321409ee17/chrome/android/java/res/layout/icon_row_menu_footer.xml
[modify] https://crrev.com/7601678d95b575aab8d337d26559fc321409ee17/chrome/android/java/src/org/chromium/chrome/browser/ChromeTabbedActivity.java
[modify] https://crrev.com/7601678d95b575aab8d337d26559fc321409ee17/chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenu.java
[modify] https://crrev.com/7601678d95b575aab8d337d26559fc321409ee17/chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenuHandler.java
[add] https://crrev.com/7601678d95b575aab8d337d26559fc321409ee17/chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenuIconRowFooter.java
[modify] https://crrev.com/7601678d95b575aab8d337d26559fc321409ee17/chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenuPropertiesDelegate.java
[modify] https://crrev.com/7601678d95b575aab8d337d26559fc321409ee17/chrome/android/java_sources.gni
[modify] https://crrev.com/7601678d95b575aab8d337d26559fc321409ee17/chrome/android/javatests/src/org/chromium/chrome/browser/appmenu/AppMenuTest.java
[add] https://crrev.com/7601678d95b575aab8d337d26559fc321409ee17/chrome/android/javatests/src/org/chromium/chrome/browser/appmenu/ChromeHomeAppMenuTest.java

Status: Fixed (was: Started)
Verified in M60-60.0.3086.0 build

Sign in to add a comment