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

Side by Side Diff: chrome/browser/resources/notification_2line.html

Issue 786023002: Remove hard-coded font families in WebUI. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: The First Version. Created 6 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 <!DOCTYPE html> 1 <!DOCTYPE html>
2 <html> 2 <html>
3 <head> 3 <head>
4 <meta charset="utf-8"> 4 <meta charset="utf-8">
5 <title>$1</title> 5 <title>$1</title>
6 <link rel="stylesheet" href="chrome://resources/css/text_defaults.css">
6 <style type="text/css"> 7 <style type="text/css">
7 #title { 8 #title {
8 font-weight: bold; 9 font-weight: bold;
9 font-size: 13px; 10 font-size: 13px;
Dan Beam 2014/12/10 19:49:25 i guess this should stay...
10 font-family: helvetica, arial, sans-serif;
11 } 11 }
12 #description { 12 #description {
13 font-family: helvetica, arial, sans-serif;
14 font-size: 13px; 13 font-size: 13px;
15 } 14 }
Dan Beam 2014/12/10 19:49:25 nit: #title, #description { font-size: 13
xdai1 2014/12/11 02:59:22 Done.
16 </style> 15 </style>
17 </head> 16 </head>
18 <body dir="$3"> 17 <body dir="$3">
Dan Beam 2014/12/10 19:49:25 dir="$3" should probably be on <html>
jungshik at Google 2014/12/10 20:46:57 I wonder whether the direction for 'notification'
xdai1 2014/12/11 02:59:23 Moved to html.
19 <div id="title">$1</div> 18 <div id="title">$1</div>
20 <div id="description">$2</div> 19 <div id="description">$2</div>
21 </body> 20 </body>
22 </html> 21 </html>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698