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

Side by Side Diff: chrome/browser/resources/pdf/index-material.html

Issue 806633003: Implement basic toolbar with Material Design and loading progress. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Tidying Created 5 years, 11 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
« no previous file with comments | « chrome/browser/resources/pdf/index-material.css ('k') | chrome/browser/resources/pdf/pdf.js » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <!doctype html> 1 <!doctype html>
2 <html i18n-values="dir:textdirection"> 2 <html i18n-values="dir:textdirection">
3 <head> 3 <head>
4 <meta charset="utf-8"> 4 <meta charset="utf-8">
5 <link rel="import" href="chrome://resources/polymer/polymer/polymer.html"> 5 <link rel="import" href="chrome://resources/polymer/polymer/polymer.html">
6 <link rel="import" href="html_office/elements/viewer-button/viewer-button.html "> 6 <link rel="import" href="html_office/elements/viewer-button/viewer-button.html ">
7 <link rel="import" href="html_office/elements/viewer-error-screen/viewer-error -screen.html"> 7 <link rel="import" href="html_office/elements/viewer-error-screen/viewer-error -screen.html">
8 <link rel="import" href="html_office/elements/viewer-page-indicator/viewer-pag e-indicator.html"> 8 <link rel="import" href="html_office/elements/viewer-page-indicator/viewer-pag e-indicator.html">
9 <link rel="import" href="html_office/elements/viewer-password-screen/viewer-pa ssword-screen.html"> 9 <link rel="import" href="html_office/elements/viewer-password-screen/viewer-pa ssword-screen.html">
10 <link rel="import" href="html_office/elements/viewer-progress-bar/viewer-progr ess-bar.html">
11 <link rel="import" href="html_office/elements/viewer-toolbar/viewer-toolbar.ht ml"> 10 <link rel="import" href="html_office/elements/viewer-toolbar/viewer-toolbar.ht ml">
11
12 <link rel="import" href="chrome://resources/polymer/core-toolbar/core-toolbar. html">
13 <link rel="import" href="chrome://resources/polymer/paper-icon-button/paper-ic on-button.html">
14 <link rel="import" href="chrome://resources/polymer/paper-progress/paper-progr ess.html">
15 <link rel="import" href="chrome://resources/polymer/paper-shadow/paper-shadow. html">
16
17 <!-- TODO(alexandrec): When https://codereview.chromium.org/846833002/ is land ed, remove this.-->
Sam McNally 2015/01/19 00:09:36 I think you can remove this now.
Alexandre Carlton 2015/01/19 00:40:12 Done.
18 <link href="http://fonts.googleapis.com/css?family=RobotoDraft:regular,bold,it alic,thin,light,bolditalic,black,medium&lang=en" rel="stylesheet" type="text/css ">
12 <link rel="stylesheet" href="chrome://resources/css/text_defaults.css"> 19 <link rel="stylesheet" href="chrome://resources/css/text_defaults.css">
13 <link rel="stylesheet" type="text/css" href="index.css"> 20 <link rel="stylesheet" type="text/css" href="index.css">
21 <link rel="stylesheet" type="text/css" href="index-material.css">
14 </head> 22 </head>
15 <body> 23 <body>
24 <paper-shadow id="toolbar-shadow" z="1">
25 <paper-progress value="0" id="progress-bar"></paper-progress>
26 <core-toolbar id="pdf-toolbar">
27 <span id="left-toolbar" flex>
Sam McNally 2015/01/19 00:09:36 Is this containing span necessary?
Alexandre Carlton 2015/01/19 00:40:12 Removed.
28 <span id="title">PDF Viewer</span>
Sam McNally 2015/01/19 00:09:36 Is "PDF Viewer" meant to be seen? If not, leave th
raymes 2015/01/19 00:11:49 +1
Alexandre Carlton 2015/01/19 00:40:12 Done.
29 </span>
30
31 <span id="middle-toolbar" flex>
Sam McNally 2015/01/19 00:09:36 Does this need an id?
Alexandre Carlton 2015/01/19 00:40:12 Removed.
32 </span>
33
34 <span id="left-toolbar">
Sam McNally 2015/01/19 00:09:36 Is this containing span necessary?
Alexandre Carlton 2015/01/19 00:40:12 Removed.
35 <paper-icon-button icon="file-download" id="save-button"></paper-icon-butt on>
36 <paper-icon-button icon="print" id="print-button"></paper-icon-button>
37 </span>
38 </core-toolbar>
39 </paper-shadow>
16 40
17 <div id="sizer"></div> 41 <div id="sizer"></div>
18 <viewer-password-screen id="password-screen"></viewer-password-screen> 42 <viewer-password-screen id="password-screen"></viewer-password-screen>
19 <viewer-page-indicator id="page-indicator"></viewer-page-indicator> 43 <viewer-page-indicator id="page-indicator"></viewer-page-indicator>
20 <viewer-progress-bar id="progress-bar"></viewer-progress-bar>
21 44
22 <viewer-toolbar id="toolbar"> 45 <viewer-toolbar id="toolbar">
23 <viewer-button id="fit-to-page-button" 46 <viewer-button id="fit-to-page-button"
24 assetpath="html_office/elements/viewer-button/" 47 assetpath="html_office/elements/viewer-button/"
25 img="button_fit_page.png" latchable> 48 img="button_fit_page.png" latchable>
26 </viewer-button> 49 </viewer-button>
27 <viewer-button id="fit-to-width-button" 50 <viewer-button id="fit-to-width-button"
28 assetpath="html_office/elements/viewer-button/" 51 assetpath="html_office/elements/viewer-button/"
29 img="button_fit_width.png" latchable> 52 img="button_fit_width.png" latchable>
30 </viewer-button> 53 </viewer-button>
31 <viewer-button id="zoom-out-button" 54 <viewer-button id="zoom-out-button"
32 assetpath="html_office/elements/viewer-button/" 55 assetpath="html_office/elements/viewer-button/"
33 img="button_zoom_out.png"> 56 img="button_zoom_out.png">
34 </viewer-button> 57 </viewer-button>
35 <viewer-button id="zoom-in-button" 58 <viewer-button id="zoom-in-button"
36 assetpath="html_office/elements/viewer-button/" 59 assetpath="html_office/elements/viewer-button/"
37 img="button_zoom_in.png"> 60 img="button_zoom_in.png">
38 </viewer-button> 61 </viewer-button>
39 <viewer-button id="save-button"
40 assetpath="html_office/elements/viewer-button/"
41 img="button_save.png">
42 </viewer-button>
43 <viewer-button id="print-button"
44 assetpath="html_office/elements/viewer-button/"
45 img="button_print.png">
46 </viewer-button>
47 </viewer-toolbar> 62 </viewer-toolbar>
48 63
49 <viewer-error-screen id="error-screen"></viewer-error-screen> 64 <viewer-error-screen id="error-screen"></viewer-error-screen>
50 65
51 </body> 66 </body>
52 <script src="main.js"></script> 67 <script src="main.js"></script>
53 <script src="pdf.js"></script> 68 <script src="pdf.js"></script>
54 <script src="viewport.js"></script> 69 <script src="viewport.js"></script>
55 <script src="open_pdf_params_parser.js"></script> 70 <script src="open_pdf_params_parser.js"></script>
56 <script src="pdf_scripting_api.js"></script> 71 <script src="pdf_scripting_api.js"></script>
57 <script src="chrome://resources/js/util.js"></script> 72 <script src="chrome://resources/js/util.js"></script>
58 </html> 73 </html>
OLDNEW
« no previous file with comments | « chrome/browser/resources/pdf/index-material.css ('k') | chrome/browser/resources/pdf/pdf.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698