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

Issue 12211124: Adds the new folder menu item to the gear button menu. (Closed)

Created:
7 years, 10 months ago by mtomasz
Modified:
7 years, 10 months ago
Reviewers:
Haruki Sato
CC:
chromium-reviews, rginda+watch_chromium.org, arv+watch_chromium.org
Visibility:
Public.

Description

Adds the new folder menu item to the gear button menu. Before, it was only in the context menu, what was not so user friendly. This patch adds this item on top of the gear menu to improve its visibility. TEST=Enter Files.app, click on the gear menu. BUG=174623 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=181880

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -1 line) Patch
M chrome/browser/resources/file_manager/main.html View 1 chunk +3 lines, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
mtomasz
@haruki: PTAL at this tiny patch. Thanks.
7 years, 10 months ago (2013-02-12 05:06:27 UTC) #1
Haruki Sato
lgtm It's disabled when unavailable, right? (e.g. read-only volume)
7 years, 10 months ago (2013-02-12 05:58:13 UTC) #2
mtomasz
On 2013/02/12 05:58:13, Haruki Sato wrote: > lgtm > > It's disabled when unavailable, right? ...
7 years, 10 months ago (2013-02-12 06:02:12 UTC) #3
commit-bot: I haz the power
7 years, 10 months ago (2013-02-12 06:02:42 UTC) #4

Powered by Google App Engine
This is Rietveld 408576698