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

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

Issue 9689007: Downloads: First pass at a cleanup to match webdev style. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Added css. Created 8 years, 9 months 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 | Annotate | Revision Log
OLDNEW
(Empty)
1 <!DOCTYPE HTML>
2 <html i18n-values="dir:textdirection;">
3 <head>
4 <meta charset="utf-8">
5 <title i18n-content="title"></title>
6 <link rel="icon" href="../../../app/theme/downloads_favicon.png">
csilv 2012/03/12 18:16:50 chrome://theme/IDR_DOWNLOADS_FAVICON
James Hawkins 2012/03/12 19:51:54 Done.
7 <link rel="stylesheet" href="../old_webui.css">
8 <link rel="stylesheet" href="../shared/css/chrome_shared.css">
9 <link rel="stylesheet" href="../shared/css/widgets.css">
10 <link rel="stylesheet" href="downloads.css">
11 <script src="chrome://resources/js/local_strings.js"></script>
12 <script src="chrome://resources/js/util.js"></script>
13 <script src="chrome://downloads/downloads.js"></script>
Dan Beam 2012/03/12 08:38:48 why aren't you indenting in <head>?
James Hawkins 2012/03/12 18:33:45 Done.
14 </head>
15 <body i18n-values=".style.fontFamily:fontfamily;.style.fontSize:fontsize">
16 <div class="header">
17 <a id="search-link" href="">
18 <img src="../shared/images/downloads_section.png"
19 width="67" height="67" class="logo" border="0"></a>
Dan Beam 2012/03/12 08:38:48 wrap </a> to next line
James Hawkins 2012/03/12 18:33:45 Done.
20 <form id="search-form" method="post" action="" class="form">
21 <input type="text" name="term" id="term">
22 <input id="search-submit" type="submit" name="submit"
23 i18n-values="value:searchbutton">
24 </form>
25 </div>
26 <div class="main">
27 <div id="downloads-summary">
28 <span id="downloads-summary-text" i18n-content="downloads"></span>
29 <span id="downloads-actions">
30 <a id="open-downloads-folder" href=""
31 i18n-content="open_downloads_folder"></a>
Dan Beam 2012/03/12 08:38:48 and here
James Hawkins 2012/03/12 18:33:45 Done.
32 <a id="clear-all" href="" i18n-content="clear_all"></a>
33 </span>
34 </div>
35 <div id="downloads-display"></div>
36 </div>
37 <script src="chrome://downloads/strings.js"></script>
38 <script src="chrome://resources/js/i18n_template.js"></script>
39 <script src="chrome://resources/js/i18n_process.js"></script>
40 </body>
41 </html>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698