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

Unified Diff: chrome/browser/resources/settings/search_engines_page/search_engine_entry.html

Issue 2529543002: MD Settings: indicate extension controlled search engines (Closed)
Patch Set: circumvent test failures Created 4 years 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/search_engines_page/search_engine_entry.html
diff --git a/chrome/browser/resources/settings/search_engines_page/search_engine_entry.html b/chrome/browser/resources/settings/search_engines_page/search_engine_entry.html
index 81130ec9cf383ca3ec5d125552bf3f969864d949..a721fe3a11d6157de7bb9baf83d9c673fe36075e 100644
--- a/chrome/browser/resources/settings/search_engines_page/search_engine_entry.html
+++ b/chrome/browser/resources/settings/search_engines_page/search_engine_entry.html
@@ -3,6 +3,7 @@
<link rel="import" href="chrome://resources/html/polymer.html">
<link rel="import" href="chrome://resources/html/icon.html">
<link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper-icon-button.html">
+<link rel="import" href="/controls/extension_controlled_indicator.html">
<link rel="import" href="/search_engines_page/search_engine_dialog.html">
<link rel="import" href="/search_engines_page/search_engine_entry_css.html">
<link rel="import" href="/search_engines_page/search_engines_browser_proxy.html">
@@ -11,7 +12,7 @@
<dom-module id="settings-search-engine-entry">
<template>
<style include="settings-shared search-engine-entry">
- :host([is-default]) {
+ :host([is-default]) .list-item {
font-weight: 500;
}
@@ -26,6 +27,11 @@
text-overflow: ellipsis;
white-space: nowrap;
}
+
+ :host(:not([show-dots_])) paper-icon-button {
+ pointer-events: none;
+ visibility: hidden;
dpapad 2016/12/01 19:02:15 Does that ensure that the button can't be reached
Dan Beam 2016/12/06 02:32:12 yes, AND fun fact: it also passes the a11y audit (
+ }
</style>
<template is="dom-if" if="[[showEditSearchEngineDialog_]]" restamp>
@@ -56,6 +62,13 @@
id="delete">$i18n{searchEnginesRemoveFromList}</button>
</dialog>
</div>
+ <template is="dom-if" if="[[engine.extension]]">
+ <extension-controlled-indicator
+ extension-id="[[engine.extension.id]]"
+ extension-name="[[engine.extension.name]]"
+ extension-can-be-disabled="[[engine.extension.canBeDisabled]]">
+ </extension-controlled-indicator>
+ </template>
</template>
<script src="search_engine_entry.js"></script>
</dom-module>

Powered by Google App Engine
This is Rietveld 408576698