Issue 10221035: FileBrowser: tuned colors according to mocks.

Issue 10221035: FileBrowser: tuned colors according to mocks. (Closed)

Created:
8 years, 7 months ago by Dmitry Zvorygin

Modified:
8 years, 7 months ago

Reviewers:
Vladislav Kaznacheev

CC:
chromium-reviews, rginda+watch_chromium.org, arv (Not doing code reviews)

Base URL:
svn://svn.chromium.org/chrome/trunk/src

Visibility:
Public.

More Reviews

Description

FileBrowser: tuned colors according to mocks. BUG=chromium-os:29969 TEST= Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=134477

Patch Set 2 : Fixed root-items border color to #aaa #

Total comments: 2

Patch Set 3 : Fixed line-height in grid view. #

Patch Set 4 : Merged with master. #

Created: 8 years, 7 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+61 lines, -59 lines)			Patch
M	chrome/browser/resources/file_manager/css/file_manager.css	View	1 2 3	4 chunks	+54 lines, -52 lines	0 comments	Download
M	chrome/browser/resources/file_manager/js/file_manager.js	View	1 2 3	5 chunks	+5 lines, -5 lines	0 comments	Download
M	chrome/browser/resources/file_manager/main.html	View	1 2 3	2 chunks	+2 lines, -2 lines	0 comments	Download

Messages

Total messages: 4 (0 generated)

Expand Messages | Collapse Messages

Vladislav Kaznacheev

https://chromiumcodereview.appspot.com/10221035/diff/2001/chrome/browser/resources/file_manager/css/file_manager.css File chrome/browser/resources/file_manager/css/file_manager.css (right): https://chromiumcodereview.appspot.com/10221035/diff/2001/chrome/browser/resources/file_manager/css/file_manager.css#newcode828 chrome/browser/resources/file_manager/css/file_manager.css:828: #list-container li { This style will be applied to ...

8 years, 7 months ago (2012-04-28 14:07:53 UTC) #2

Dmitry Zvorygin

PTAL. https://chromiumcodereview.appspot.com/10221035/diff/2001/chrome/browser/resources/file_manager/css/file_manager.css File chrome/browser/resources/file_manager/css/file_manager.css (right): https://chromiumcodereview.appspot.com/10221035/diff/2001/chrome/browser/resources/file_manager/css/file_manager.css#newcode828 chrome/browser/resources/file_manager/css/file_manager.css:828: #list-container li { On 2012/04/28 14:07:53, Vladislav Kaznacheev ...

8 years, 7 months ago (2012-04-28 14:21:55 UTC) #3

lgtm

Expand Messages | Collapse Messages