Chromium Code Reviews| OLD | NEW |
|---|---|
| (Empty) | |
| 1 <link rel="import" href="chrome://resources/html/polymer.html"> | |
| 2 | |
| 3 <link rel="import" href="chrome://resources/cr_elements/cr_lazy_render/cr_lazy_r ender.html"> | |
| 4 <link rel="import" href="chrome://resources/polymer/v1_0/paper-toast/paper-toast .html"> | |
| 5 <link rel="import" href="chrome://resources/polymer/v1_0/paper-button/paper-butt on.html"> | |
| 6 <link rel="import" href="chrome://bookmarks/shared_style.html"> | |
| 7 | |
| 8 <dom-module id="bookmarks-toast-manager"> | |
| 9 <template> | |
| 10 <style include="shared-style"> | |
| 11 paper-toast { | |
| 12 align-items: center; | |
| 13 display: flex; | |
| 14 height: 52px; | |
| 15 max-width: 568px; | |
|
tsergeant
2017/05/26 05:19:24
A couple of additions to match the mocks:
font-si
calamity
2017/05/31 08:10:41
Pushed into cr-toast.
| |
| 16 padding: 0 16px 0 24px; | |
| 17 } | |
| 18 | |
| 19 #button-container { | |
| 20 align-items: center; | |
| 21 display: flex; | |
| 22 height: 0; | |
|
tsergeant
2017/05/26 05:19:24
Is this necessary?
calamity
2017/05/31 08:10:41
Whole thing wasn't necessary actually. Removed.
| |
| 23 } | |
| 24 | |
| 25 #content { | |
| 26 flex: 1; | |
| 27 margin-right: 32px; | |
| 28 overflow: hidden; | |
| 29 text-overflow: ellipsis; | |
| 30 white-space: nowrap; | |
| 31 } | |
| 32 | |
| 33 paper-button { | |
| 34 color: var(--google-blue-300); | |
| 35 font-size: 13px; | |
|
tsergeant
2017/05/26 05:19:24
With font-size: inherit above, you won't need to a
calamity
2017/05/31 08:10:41
Acknowledged.
| |
| 36 font-weight: 500; | |
| 37 height: 32px; | |
| 38 margin: 0; | |
| 39 min-width: 52px; | |
| 40 padding: 0; | |
|
tsergeant
2017/05/26 05:19:24
I think the mocks call for padding: 8px here.
It
calamity
2017/05/31 08:10:41
Done.
| |
| 41 } | |
| 42 </style> | |
| 43 <template is="cr-lazy-render" id="toast"> | |
| 44 <paper-toast duration="5000"> | |
| 45 <div id="content"></div> | |
| 46 <div id="button-container"> | |
| 47 <paper-button id="button" hidden$="[[!showUndo_]]" | |
| 48 on-tap="onUndoTap_"> | |
| 49 $i18n{undo} | |
| 50 </paper-button> | |
| 51 </div> | |
| 52 </paper-toast> | |
| 53 </template> | |
| 54 </template> | |
| 55 <script src="chrome://bookmarks/toast_manager.js"></script> | |
| 56 </dom-module> | |
| OLD | NEW |