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

Unified Diff: chrome/browser/resources/settings/site_settings/protocol_handlers.html

Issue 2377333002: Site Settings Desktop: Use 'actionable' more throughout. (Closed)
Patch Set: Created 4 years, 3 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
« no previous file with comments | « no previous file | chrome/browser/resources/settings/site_settings/site_list.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/settings/site_settings/protocol_handlers.html
diff --git a/chrome/browser/resources/settings/site_settings/protocol_handlers.html b/chrome/browser/resources/settings/site_settings/protocol_handlers.html
index 8b88ccc9587633146b176e309de921368d9a8eeb..ab78ee849ffd445172499102b21594cc8b55584d 100644
--- a/chrome/browser/resources/settings/site_settings/protocol_handlers.html
+++ b/chrome/browser/resources/settings/site_settings/protocol_handlers.html
@@ -53,14 +53,15 @@
<paper-icon-button icon="cr:more-vert"
class="dropdown-trigger">
</paper-icon-button>
- <paper-menu id="actionMenu" class="dropdown-content"
+ <paper-menu id="actionMenu" class="dropdown-content" actionable
on-iron-activate="onActionMenuIronActivate_"
attr-for-selected="menu-value">
<paper-item menu-value$="[[menuActions_.SET_DEFAULT]]"
- hidden$="[[isDefault_(index, protocol.default_handler)]]">
+ hidden$="[[isDefault_(index, protocol.default_handler)]]"
+ actionable>
Dan Beam 2016/09/29 17:41:14 damn, I didn't add actionable in these places beca
$i18n{handlerSetDefault}
</paper-item>
- <paper-item menu-value$="[[menuActions_.REMOVE]]">
+ <paper-item menu-value$="[[menuActions_.REMOVE]]" actionable>
$i18n{handlerRemove}
</paper-item>
</paper-menu>
« no previous file with comments | « no previous file | chrome/browser/resources/settings/site_settings/site_list.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698