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

Side by Side Diff: chrome/browser/resources/settings/on_startup_page/startup_urls_page.html

Issue 1869883003: MD Settings: On Startup, extract dialog to its own file. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add test. Created 4 years, 8 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
OLDNEW
1 <link rel="import" href="chrome://resources/html/polymer.html"> 1 <link rel="import" href="chrome://resources/html/polymer.html">
2 <link rel="import" href="chrome://resources/html/util.html"> 2 <link rel="import" href="chrome://resources/html/util.html">
3 <link rel="import" href="chrome://resources/html/web_ui_listener_behavior.html"> 3 <link rel="import" href="chrome://resources/html/web_ui_listener_behavior.html">
4 <link rel="import" href="chrome://resources/polymer/v1_0/paper-input/paper-input .html">
5 <link rel="import" href="chrome://md-settings/settings_dialog.html">
6 <link rel="import" href="chrome://md-settings/settings_shared_css.html"> 4 <link rel="import" href="chrome://md-settings/settings_shared_css.html">
5 <link rel="import" href="chrome://md-settings/on_startup_page/startup_url_dialog .html">
7 <link rel="import" href="chrome://md-settings/on_startup_page/startup_urls_page_ browser_proxy.html"> 6 <link rel="import" href="chrome://md-settings/on_startup_page/startup_urls_page_ browser_proxy.html">
8 7
9 <dom-module id="settings-startup-urls-page"> 8 <dom-module id="settings-startup-urls-page">
10 <link rel="import" type="css" href="on_startup_shared.css"> 9 <link rel="import" type="css" href="on_startup_shared.css">
11 <template> 10 <template>
12 <style include="settings-shared"> 11 <style include="settings-shared">
13 .favicon-image { 12 .favicon-image {
14 background-size: contain; 13 background-size: contain;
15 background-repeat: no-repeat; 14 background-repeat: no-repeat;
16 height: 16px; 15 height: 16px;
17 width: 16px; 16 width: 16px;
18 } 17 }
19 </style> 18 </style>
20 <div class="list-frame vertical-list"> 19 <div class="list-frame vertical-list">
21 <template is="dom-repeat" items="[[startupPages_]]"> 20 <template is="dom-repeat" items="[[startupPages_]]">
22 <div class="list-item"> 21 <div class="list-item">
23 <div class="favicon-image" 22 <div class="favicon-image"
24 style="background-image: [[getIconSet_(item.url)]]"> 23 style="background-image: [[getIconSet_(item.url)]]">
25 </div> 24 </div>
26 <div class="middle"> 25 <div class="middle">
27 <div class="text-elide">[[item.title]]</div> 26 <div class="text-elide">[[item.title]]</div>
28 <div class="text-elide secondary">[[item.url]]</div> 27 <div class="text-elide secondary">[[item.url]]</div>
29 </div> 28 </div>
30 <iron-icon on-tap="onRemoveUrlTap_" icon="clear"></iron-icon> 29 <iron-icon on-tap="onRemoveUrlTap_" icon="clear"></iron-icon>
31 </div> 30 </div>
32 </template> 31 </template>
33 <div class="list-item list-button" id="addPage" on-tap="onAddPageTap_"> 32 <div class="list-item list-button" id="addPage" on-tap="onAddPageTap_">
34 $i18n{onStartupAddNewPage} 33 $i18n{onStartupAddNewPage}
35 </div> 34 </div>
36 <div class="list-item list-button" on-tap="onUseCurrentPagesTap_"> 35 <div class="list-item list-button" id="useCurrentPages"
37 $i18n{onStartupUseCurrent} 36 on-tap="onUseCurrentPagesTap_">$i18n{onStartupUseCurrent}</div>
38 </div>
39 </div> 37 </div>
40 38 <template is="dom-if" if="[[showStartupUrlDialog_]]" restamp>
Dan Beam 2016/04/08 19:20:04 you don't really need to move this just to wrap it
41 <settings-dialog id="addUrlDialog"> 39 <settings-startup-url-dialog></settings-startup-url-dialog>
42 <div class="title">$i18n{onStartupAddNewPage}</div> 40 </template>
43 <div class="body">
44 <paper-input class="flex" always-float-label id="newUrl"
45 label="$i18n{onStartupSiteUrl}" value="{{newUrl_}}">
46 </paper-input>
47 </div>
48 <div class="button-container">
49 <div class="action-buttons">
50 <paper-button class="cancel-button" on-tap="onCancelTap_"
51 id="cancel">$i18n{cancel}</paper-button>
52 <paper-button id="add" class="action-button" on-tap="onAddTap_"
53 disabled="[[!isNewUrlValid_]]">$i18n{add}</paper-button>
54 </div>
55 </div>
56 </settings-dialog>
57 </template> 41 </template>
58 <script src="startup_urls_page.js"></script> 42 <script src="startup_urls_page.js"></script>
59 </dom-module> 43 </dom-module>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698