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

Side by Side 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, 2 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 | « no previous file | chrome/browser/resources/settings/site_settings/site_list.html » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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/cr_elements/icons.html"> 2 <link rel="import" href="chrome://resources/cr_elements/icons.html">
3 <link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper -icon-button.html"> 3 <link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper -icon-button.html">
4 <link rel="import" href="chrome://resources/polymer/v1_0/paper-item/paper-item.h tml"> 4 <link rel="import" href="chrome://resources/polymer/v1_0/paper-item/paper-item.h tml">
5 <link rel="import" href="chrome://resources/polymer/v1_0/paper-menu/paper-menu.h tml"> 5 <link rel="import" href="chrome://resources/polymer/v1_0/paper-menu/paper-menu.h tml">
6 <link rel="import" href="chrome://resources/polymer/v1_0/paper-menu-button/paper -menu-button.html"> 6 <link rel="import" href="chrome://resources/polymer/v1_0/paper-menu-button/paper -menu-button.html">
7 <link rel="import" href="chrome://resources/polymer/v1_0/paper-toggle-button/pap er-toggle-button.html"> 7 <link rel="import" href="chrome://resources/polymer/v1_0/paper-toggle-button/pap er-toggle-button.html">
8 <link rel="import" href="chrome://resources/html/web_ui_listener_behavior.html"> 8 <link rel="import" href="chrome://resources/html/web_ui_listener_behavior.html">
9 <link rel="import" href="/i18n_setup.html"> 9 <link rel="import" href="/i18n_setup.html">
10 <link rel="import" href="/settings_shared_css.html"> 10 <link rel="import" href="/settings_shared_css.html">
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
46 <div class="secondary" 46 <div class="secondary"
47 hidden$="[[!isDefault_(index, protocol.default_handler)]]"> 47 hidden$="[[!isDefault_(index, protocol.default_handler)]]">
48 $i18n{handlerIsDefault} 48 $i18n{handlerIsDefault}
49 </div> 49 </div>
50 </div> 50 </div>
51 51
52 <paper-menu-button> 52 <paper-menu-button>
53 <paper-icon-button icon="cr:more-vert" 53 <paper-icon-button icon="cr:more-vert"
54 class="dropdown-trigger"> 54 class="dropdown-trigger">
55 </paper-icon-button> 55 </paper-icon-button>
56 <paper-menu id="actionMenu" class="dropdown-content" 56 <paper-menu id="actionMenu" class="dropdown-content" actionable
57 on-iron-activate="onActionMenuIronActivate_" 57 on-iron-activate="onActionMenuIronActivate_"
58 attr-for-selected="menu-value"> 58 attr-for-selected="menu-value">
59 <paper-item menu-value$="[[menuActions_.SET_DEFAULT]]" 59 <paper-item menu-value$="[[menuActions_.SET_DEFAULT]]"
60 hidden$="[[isDefault_(index, protocol.default_handler)]]"> 60 hidden$="[[isDefault_(index, protocol.default_handler)]]"
61 actionable>
Dan Beam 2016/09/29 17:41:14 damn, I didn't add actionable in these places beca
61 $i18n{handlerSetDefault} 62 $i18n{handlerSetDefault}
62 </paper-item> 63 </paper-item>
63 <paper-item menu-value$="[[menuActions_.REMOVE]]"> 64 <paper-item menu-value$="[[menuActions_.REMOVE]]" actionable>
64 $i18n{handlerRemove} 65 $i18n{handlerRemove}
65 </paper-item> 66 </paper-item>
66 </paper-menu> 67 </paper-menu>
67 </paper-menu-button> 68 </paper-menu-button>
68 </div> 69 </div>
69 70
70 </template> 71 </template>
71 </div> 72 </div>
72 </template> 73 </template>
73 </template> 74 </template>
74 <script src="protocol_handlers.js"></script> 75 <script src="protocol_handlers.js"></script>
75 </dom-module> 76 </dom-module>
OLDNEW
« 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