Chromium Code Reviews Chromium Code Reviews
|
| OLD | NEW |
|---|---|
| (Empty) | |
| 1 <!DOCTYPE HTML> | |
| 2 <html i18n-values="dir:textdirection;" id="t"> | |
|
arv (Not doing code reviews)
2010/10/06 21:53:10
You can remove the id here
| |
| 3 <head> | |
| 4 <meta charset="utf-8"> | |
| 5 <title i18n-content="title"></title> | |
| 6 | |
| 7 <link rel="stylesheet" href="dom_ui.css"> | |
| 8 <link rel="stylesheet" href="print_preview.css"> | |
| 9 | |
| 10 <script src="chrome://resources/js/local_strings.js"></script> | |
| 11 <script src="chrome://resources/js/util.js"></script> | |
| 12 | |
| 13 </head> | |
| 14 <body i18n-values=".style.fontFamily:fontfamily;.style.fontSize:fontsize"> | |
| 15 <div id="main-content"> | |
|
arv (Not doing code reviews)
2010/10/06 21:53:10
Any reason why you need this div?
| |
| 16 <div id="navbar-container"> | |
| 17 <div id="destination-container"> | |
| 18 <div id="destination-title"> | |
|
arv (Not doing code reviews)
2010/10/06 21:53:10
Looks like a<hx> tag would be more suitable
| |
| 19 Destination | |
| 20 </div> | |
| 21 <div id="destination-list"> | |
| 22 LIST GOES HERE | |
| 23 </div> | |
| 24 <div id="print-buttons"> | |
| 25 <button id="printButton">Print</button> | |
| 26 <button id="cancelButton">Cancel</button> | |
| 27 </div> | |
| 28 </div> | |
| 29 <div id="options-container"> | |
| 30 OPTIONS HERE | |
| 31 </div> | |
| 32 </div> | |
| 33 <div id="mainview"> | |
| 34 <div id="mainview-content"> | |
|
arv (Not doing code reviews)
2010/10/06 21:53:10
I think you can remove all the *-content divs.
| |
| 35 MAIN | |
| 36 </div> | |
| 37 </div> | |
| 38 <div id="thumbnails"> | |
| 39 <div id="thumbnails-content"> | |
| 40 THUMBNAILS | |
| 41 </div> | |
| 42 </div> | |
| 43 </div> | |
| 44 | |
| 45 <script> | |
| 46 console.log('in bottom script'); | |
| 47 </script> | |
| 48 </body> | |
| 49 </html> | |
| OLD | NEW |
