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

Issue 6369006: css changes for clear browsing data page. (Closed)

Created:
9 years, 11 months ago by Evan Stade
Modified:
9 years, 6 months ago
CC:
chromium-reviews, arv (Not doing code reviews)
Visibility:
Public.

Description

css changes for clear browsing data page. css from ainslie@ BUG=63843 TEST=visual Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=71905

Patch Set 1 #

Patch Set 2 : updates #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -8 lines) Patch
M chrome/browser/resources/options/clear_browser_data_overlay.css View 1 2 chunks +7 lines, -6 lines 0 comments Download
M chrome/browser/resources/options/options_page.css View 1 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Evan Stade
9 years, 11 months ago (2011-01-19 21:48:37 UTC) #1
stuartmorgan
LGTM
9 years, 11 months ago (2011-01-19 21:58:14 UTC) #2
ainslie
Three quick updates: 1. Add a bit of space between the ok/cancel buttons: button-strip > ...
9 years, 11 months ago (2011-01-20 00:16:05 UTC) #3
Evan Stade
9 years, 11 months ago (2011-01-20 01:13:26 UTC) #4
On 2011/01/20 00:16:05, ainslie wrote:
> Three quick updates:
> 
> 1. Add a bit of space between the ok/cancel buttons: 
> button-strip > button 
> -webkit-margin-start: 10px;

done.

> 
> 2. Shorten the link text to: 
> Adobe Flash Player local storage ...

bauerb just changed this string within the last week. I'll leave it alone for
now and ping him on if we can shorten it again.

> 
> 3. Give the dialogs a min-width of 500px. 
> (in this case clearBrowserDataOverlay)
> 
> Otherwise LGTM.

done.

Powered by Google App Engine
This is Rietveld 408576698