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

Unified 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/downloads/downloads.html
diff --git a/chrome/browser/resources/downloads/downloads.html b/chrome/browser/resources/downloads/downloads.html
new file mode 100644
index 0000000000000000000000000000000000000000..d51ec58c19b229824bcdb4fb3b063ea8d022416a
--- /dev/null
+++ b/chrome/browser/resources/downloads/downloads.html
@@ -0,0 +1,41 @@
+<!DOCTYPE HTML>
+<html i18n-values="dir:textdirection;">
+<head>
+<meta charset="utf-8">
+<title i18n-content="title"></title>
+<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.
+<link rel="stylesheet" href="../old_webui.css">
+<link rel="stylesheet" href="../shared/css/chrome_shared.css">
+<link rel="stylesheet" href="../shared/css/widgets.css">
+<link rel="stylesheet" href="downloads.css">
+<script src="chrome://resources/js/local_strings.js"></script>
+<script src="chrome://resources/js/util.js"></script>
+<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.
+</head>
+<body i18n-values=".style.fontFamily:fontfamily;.style.fontSize:fontsize">
+ <div class="header">
+ <a id="search-link" href="">
+ <img src="../shared/images/downloads_section.png"
+ 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.
+ <form id="search-form" method="post" action="" class="form">
+ <input type="text" name="term" id="term">
+ <input id="search-submit" type="submit" name="submit"
+ i18n-values="value:searchbutton">
+ </form>
+ </div>
+ <div class="main">
+ <div id="downloads-summary">
+ <span id="downloads-summary-text" i18n-content="downloads"></span>
+ <span id="downloads-actions">
+ <a id="open-downloads-folder" href=""
+ 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.
+ <a id="clear-all" href="" i18n-content="clear_all"></a>
+ </span>
+ </div>
+ <div id="downloads-display"></div>
+ </div>
+ <script src="chrome://downloads/strings.js"></script>
+ <script src="chrome://resources/js/i18n_template.js"></script>
+ <script src="chrome://resources/js/i18n_process.js"></script>
+</body>
+</html>

Powered by Google App Engine
This is Rietveld 408576698