Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 <link rel="import" href="chrome://resources/html/polymer.html"> | 1 <link rel="import" href="chrome://resources/html/polymer.html"> |
| 2 | 2 |
| 3 <link rel="import" href="chrome://resources/cr_elements/cr_action_menu/cr_action _menu.html"> | 3 <link rel="import" href="chrome://resources/cr_elements/cr_action_menu/cr_action _menu.html"> |
| 4 <link rel="import" href="chrome://resources/cr_elements/cr_lazy_render/cr_lazy_r ender.html"> | 4 <link rel="import" href="chrome://resources/cr_elements/cr_lazy_render/cr_lazy_r ender.html"> |
| 5 <link rel="import" href="chrome://resources/polymer/v1_0/iron-a11y-keys-behavior /iron-a11y-keys-behavior.html"> | 5 <link rel="import" href="chrome://resources/polymer/v1_0/iron-a11y-keys-behavior /iron-a11y-keys-behavior.html"> |
| 6 <link rel="import" href="chrome://bookmarks/edit_dialog.html"> | 6 <link rel="import" href="chrome://bookmarks/edit_dialog.html"> |
| 7 <link rel="import" href="chrome://bookmarks/shared_style.html"> | 7 <link rel="import" href="chrome://bookmarks/shared_style.html"> |
| 8 <link rel="import" href="chrome://bookmarks/store_client.html"> | 8 <link rel="import" href="chrome://bookmarks/store_client.html"> |
| 9 | 9 |
| 10 <dom-module id="bookmarks-command-manager"> | 10 <dom-module id="bookmarks-command-manager"> |
| 11 <template> | 11 <template> |
| 12 <style include="shared-style"></style> | 12 <style include="shared-style"> |
| 13 .label { | |
| 14 flex: 1; | |
| 15 } | |
| 16 | |
| 17 .sublabel { | |
| 18 -webkit-margin-start: 8px; | |
| 19 color: var(--secondary-text-color); | |
| 20 text-align: end; | |
| 21 width: 24px; | |
|
tsergeant
2017/06/16 00:25:58
Consider this optional, but maybe it would be bett
calamity
2017/06/20 03:30:39
Done.
| |
| 22 } | |
| 23 | |
| 24 :not(:host([has-any-sublabel_])).sublabel { | |
|
tsergeant
2017/06/16 00:25:58
This selector doesn't work.
I think you need some
calamity
2017/06/20 03:30:39
Done.
| |
| 25 display: none; | |
| 26 } | |
| 27 | |
| 28 button { | |
| 29 display: flex; | |
|
tsergeant
2017/06/16 00:25:58
I don't think this is necessary? .dropdown-item is
calamity
2017/06/20 03:30:39
Done.
| |
| 30 } | |
| 31 </style> | |
| 13 <dialog is="cr-action-menu" id="dropdown" on-mousedown="onMenuMousedown_"> | 32 <dialog is="cr-action-menu" id="dropdown" on-mousedown="onMenuMousedown_"> |
| 14 <template is="dom-repeat" items="[[menuCommands_]]" as="command"> | 33 <template is="dom-repeat" items="[[menuCommands_]]" as="command"> |
| 15 <button class="dropdown-item" | 34 <button class="dropdown-item" |
| 16 command$="[[command]]" | 35 command$="[[command]]" |
| 17 hidden$="[[!isCommandVisible_(command, menuIds_)]]" | 36 hidden$="[[!isCommandVisible_(command, menuIds_)]]" |
| 18 disabled$="[[!isCommandEnabled_(command, menuIds_)]]" | 37 disabled$="[[!isCommandEnabled_(command, menuIds_)]]" |
| 19 on-tap="onCommandClick_"> | 38 on-tap="onCommandClick_"> |
| 20 [[getCommandLabel_(command, menuIds_)]] | 39 <span class="label"> |
| 40 [[getCommandLabel_(command, menuIds_)]] | |
| 41 </span> | |
| 42 <span class="sublabel"> | |
| 43 [[getCommandSublabel_(command, menuIds_)]] | |
| 44 </span> | |
| 21 </button> | 45 </button> |
| 22 <hr hidden$="[[!showDividerAfter_(command)]]"></hr> | 46 <hr hidden$="[[!showDividerAfter_(command)]]"></hr> |
| 23 </template> | 47 </template> |
| 24 </dialog> | 48 </dialog> |
| 25 <template is="cr-lazy-render" id="editDialog"> | 49 <template is="cr-lazy-render" id="editDialog"> |
| 26 <bookmarks-edit-dialog></bookmarks-edit-dialog> | 50 <bookmarks-edit-dialog></bookmarks-edit-dialog> |
| 27 </template> | 51 </template> |
| 28 </template> | 52 </template> |
| 29 <script src="chrome://bookmarks/command_manager.js"></script> | 53 <script src="chrome://bookmarks/command_manager.js"></script> |
| 30 </dom-module> | 54 </dom-module> |
| OLD | NEW |