Adjust style of the popup dialogs's title in Files.app.

Adjust style of the popup dialogs's title in Files.app.

This patch makes the title no bold and 15 pixels size to match the recent specs.

TBR=yoshiki@chromium.org
TEST=Tested manually. Invoke the open with dialog on a file which has more than one handler.
BUG=239304
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=202830

[mtomasz, 2013-05-29 03:53:21]

@hirono: PTAL at this tiny patch. Thanks.

[hirono, 2013-05-29 04:03:02]

https://chromiumcodereview.appspot.com/16014003/diff/1/chrome/browser/resourc...
File chrome/browser/resources/file_manager/css/common.css (right):

https://chromiumcodereview.appspot.com/16014003/diff/1/chrome/browser/resourc...
chrome/browser/resources/file_manager/css/common.css:661: font-size: 125%;
Cannot we specify the direct value '15px'?

[mtomasz, 2013-05-29 04:05:45]

On 2013/05/29 04:03:02, hirono wrote:
>
https://chromiumcodereview.appspot.com/16014003/diff/1/chrome/browser/resourc...
> File chrome/browser/resources/file_manager/css/common.css (right):
> 
>
https://chromiumcodereview.appspot.com/16014003/diff/1/chrome/browser/resourc...
> chrome/browser/resources/file_manager/css/common.css:661: font-size: 125%;
> Cannot we specify the direct value '15px'?

Nope, the UI has to be scalable if we change the font size in
chrome://settings/search#font.

[hirono, 2013-05-29 04:09:27]

On 2013/05/29 04:05:45, mtomasz wrote:
> On 2013/05/29 04:03:02, hirono wrote:
> >
>
https://chromiumcodereview.appspot.com/16014003/diff/1/chrome/browser/resourc...
> > File chrome/browser/resources/file_manager/css/common.css (right):
> > 
> >
>
https://chromiumcodereview.appspot.com/16014003/diff/1/chrome/browser/resourc...
> > chrome/browser/resources/file_manager/css/common.css:661: font-size: 125%;
> > Cannot we specify the direct value '15px'?
> 
> Nope, the UI has to be scalable if we change the font size in
> chrome://settings/search#font.

I didn't know this. Thank you! lgtm.

[commit-bot: I haz the power, 2013-05-29 04:15:13]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a lowly provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[mtomasz, 2013-05-29 04:17:01]

On 2013/05/29 04:09:27, hirono wrote:
> On 2013/05/29 04:05:45, mtomasz wrote:
> > On 2013/05/29 04:03:02, hirono wrote:
> > >
> >
>
https://chromiumcodereview.appspot.com/16014003/diff/1/chrome/browser/resourc...
> > > File chrome/browser/resources/file_manager/css/common.css (right):
> > > 
> > >
> >
>
https://chromiumcodereview.appspot.com/16014003/diff/1/chrome/browser/resourc...
> > > chrome/browser/resources/file_manager/css/common.css:661: font-size: 125%;
> > > Cannot we specify the direct value '15px'?
> > 
> > Nope, the UI has to be scalable if we change the font size in
> > chrome://settings/search#font.
> 
> I didn't know this. Thank you! lgtm.

The current Files.app is not well written for that. I think we should even avoid
hard-coded line-height, but fixing that is a big task for the future.

[commit-bot: I haz the power, 2013-05-29 04:17:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mtomasz@chromium.org/16014003/1

[commit-bot: I haz the power, 2013-05-29 08:58:35]

Change committed as 202830