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

Unified Diff: chrome/browser/resources/settings/on_startup_page/startup_url_entry.html

Issue 1876283002: MD Settings: On Startup, moving remove URL option under a dots menu. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@edit_on_startup_cpp
Patch Set: Nits. 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/settings/on_startup_page/startup_url_entry.html
diff --git a/chrome/browser/resources/settings/on_startup_page/startup_url_entry.html b/chrome/browser/resources/settings/on_startup_page/startup_url_entry.html
new file mode 100644
index 0000000000000000000000000000000000000000..509f510f11a0420d5621757f0cf24285f876de81
--- /dev/null
+++ b/chrome/browser/resources/settings/on_startup_page/startup_url_entry.html
@@ -0,0 +1,52 @@
+<link rel="import" href="chrome://resources/html/polymer.html">
+<link rel="import" href="chrome://resources/html/util.html">
+<link rel="import" href="chrome://resources/polymer/v1_0/iron-dropdown/iron-dropdown.html">
+<link rel="import" href="chrome://resources/polymer/v1_0/iron-icons/iron-icons.html">
+<link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper-icon-button.html">
+<link rel="import" href="chrome://resources/polymer/v1_0/paper-item/paper-item.html">
+<link rel="import" href="chrome://md-settings/on_startup_page/startup_urls_page_browser_proxy.html">
+<link rel="import" href="chrome://md-settings/settings_shared_css.html">
+
+<dom-module id="settings-startup-url-entry">
+ <template>
+ <style include="settings-shared">
+ .favicon-image {
+ background-size: contain;
+ background-repeat: no-repeat;
+ height: 16px;
+ width: 16px;
+ }
+
+ .dropdown-content {
+ background: white;
+ box-shadow: 0 2px 6px grey;
+ }
+
+ paper-item:hover {
+ background-color: var(--settings-hover-color);
+ }
+ </style>
+ <div class="list-item">
Dan Beam 2016/04/12 18:23:26 to fix the grey lines, move class="list-item" onto
+ <div class="favicon-image"
+ style="background-image: [[getIconSet_(model.url)]]">
+ </div>
+ <div class="middle">
+ <div class="text-elide">[[model.title]]</div>
+ <div class="text-elide secondary">[[model.url]]</div>
+ </div>
+ <paper-icon-button id="dots" icon="more-vert" toggles
+ active="{{menuOpened}}"></paper-icon-button>
+ <template is="dom-if" if="[[menuOpened]]">
+ <iron-dropdown vertical-align="top" horizontal-align="right"
+ opened="{{menuOpened}}">
+ <div class="dropdown-content">
+ <paper-item id="remove" on-tap="onRemoveTap_">
+ $i18n{onStartupRemove}
+ </paper-item>
+ </div>
+ </iron-dropdown>
+ </template>
+ </div>
+ </template>
+ <script src="startup_url_entry.js"></script>
+</dom-module>

Powered by Google App Engine
This is Rietveld 408576698