Chromium Code Reviews
Help | Chromium Project | Sign in
(416)

Issue 5640001: Polish the about:conflicts page.... (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
3 years, 4 months ago by Finnur
Modified:
2 years, 10 months ago
Reviewers:
Nico
CC:
chromium-reviews_chromium.org, arv, ben+cc_chromium.org
Visibility:
Public.

Description

Polish the about:conflicts page.

Address review comments from Nico and add a separating line as
requested by Glen.

BUG=51105
TEST=None


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=68346

Patch Set 1 #

Total comments: 4
Unified diffs Side-by-side diffs Delta from patch set Stats (+38 lines, -18 lines) Lint Patch
M chrome/browser/resources/about_conflicts.html View 2 chunks +38 lines, -18 lines 4 comments 0 errors Download
Commit:

Messages

Total messages: 3
Finnur
3 years, 4 months ago #1
Nico
the datacell stuff LG if you want to submit this in a separate CL http://codereview.chromium.org/5640001/diff/1/chrome/browser/resources/about_conflicts.html ...
3 years, 4 months ago #2
Finnur
3 years, 4 months ago #3
http://codereview.chromium.org/5640001/diff/1/chrome/browser/resources/about_...
File chrome/browser/resources/about_conflicts.html (right):

http://codereview.chromium.org/5640001/diff/1/chrome/browser/resources/about_...
chrome/browser/resources/about_conflicts.html:155: }
I tried many variations of this and couldn't get it to work...

On 2010/12/03 18:51:01, Nico wrote:
> does
> 
> .extra-info-block > * { display:block; }
> 
> work instead of this as well?

http://codereview.chromium.org/5640001/diff/1/chrome/browser/resources/about_...
chrome/browser/resources/about_conflicts.html:277:
i18n-content="helpCenterLink">HELP_CENTER</span></a>
Nope, I don't need the span. Removed. Thanks.

On 2010/12/03 18:51:01, Nico wrote:
> do you need the span in the a, or can you put the i18n-content in the a
> directly?
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1280:2d3e6564b7b6