Chromium Code Reviews| OLD | NEW |
|---|---|
| (Empty) | |
| 1 <link rel="import" href="chrome://resources/polymer/v1_0/polymer/polymer.html"> | |
| 2 <link rel="import" href="chrome://resources/polymer/v1_0/paper-styles/shadow.htm l"> | |
| 3 <link rel="import" href="chrome://resources/polymer/v1_0/iron-collapse/iron-coll apse.html"> | |
| 4 <link rel="import" href="chrome://resources/polymer/v1_0/iron-icon/iron-icon.htm l"> | |
| 5 <link rel="import" href="chrome://resources/polymer/v1_0/iron-icons/iron-icons.h tml"> | |
| 6 <link rel="import" href="chrome://resources/polymer/v1_0/iron-flex-layout/iron-f lex-layout.html"> | |
| 7 <link rel="import" href="chrome://resources/html/util.html"> | |
| 8 | |
| 9 <dom-module id="synced-device-card"> | |
| 10 <template> | |
| 11 <style> | |
| 12 :host { | |
| 13 @apply(--layout-center); | |
| 14 @apply(--layout-vertical); | |
| 15 padding: 0 24px 20px 24px; | |
| 16 } | |
| 17 | |
| 18 #main-container { | |
| 19 @apply(--shadow-elevation-2dp); | |
| 20 background: #fff; | |
| 21 border-radius: 2px; | |
| 22 max-width: 960px; | |
| 23 width: 100%; | |
| 24 } | |
| 25 | |
| 26 #tab-url { | |
| 27 -webkit-margin-end: 20px; | |
| 28 color: #333; | |
| 29 overflow: hidden; | |
| 30 text-decoration: none; | |
| 31 text-overflow: ellipsis; | |
| 32 white-space: nowrap; | |
| 33 } | |
| 34 | |
| 35 #tab-url:hover { | |
| 36 text-decoration: underline; | |
| 37 } | |
| 38 | |
| 39 #title { | |
| 40 @apply(--layout-center); | |
| 41 @apply(--layout-horizontal); | |
| 42 @apply(--layout-justified); | |
| 43 -webkit-padding-start: 20px; | |
| 44 background: #fafafa; | |
| 45 border: 0; | |
| 46 border-bottom: 1px solid rgba(0, 0, 0, 0.14); | |
| 47 border-radius: 2px 2px 0 0; | |
| 48 color: #333; | |
| 49 cursor: pointer; | |
| 50 font-size: 14px; | |
| 51 font-weight: 500; | |
| 52 height: 48px; | |
| 53 } | |
| 54 | |
| 55 .website-icon { | |
| 56 -webkit-margin-end: 16px; | |
| 57 -webkit-margin-start: 20px; | |
| 58 height: 16px; | |
| 59 min-width: 16px; | |
| 60 } | |
| 61 | |
| 62 .item-container { | |
| 63 @apply(--layout-center); | |
| 64 @apply(--layout-horizontal); | |
| 65 min-height: 40px; | |
| 66 } | |
| 67 | |
| 68 #tab-item-list { | |
| 69 padding: 8px 0; | |
| 70 } | |
| 71 | |
| 72 #open-tabs { | |
| 73 -webkit-margin-start: 20px; | |
| 74 color: rgb(102, 136, 238); | |
| 75 cursor: pointer; | |
| 76 } | |
| 77 | |
| 78 #open-tabs:hover { | |
| 79 text-decoration: underline; | |
| 80 } | |
| 81 | |
| 82 #last-update-time { | |
| 83 -webkit-padding-start: 0.3em; | |
|
calamity
2016/02/02 04:09:47
Hmm. Maybe nbsp is the right thing to use here? (T
tsergeant
2016/02/03 03:00:00
uuuuuhhhh, I'm mostly impartial. If nbsp looks the
yingran
2016/02/09 04:21:34
It adds a slightly fatter space so I'm gunna leave
| |
| 84 color: #969696; | |
| 85 } | |
| 86 | |
| 87 #dropdown-indicator { | |
| 88 -webkit-margin-end: 12px; | |
| 89 max-width: 16px; | |
| 90 } | |
| 91 | |
| 92 #collapse { | |
| 93 overflow: hidden; | |
| 94 } | |
|
calamity
2016/02/02 04:09:47
Too much shared style!
Consider: https://www.poly
yingran
2016/02/09 04:21:34
Done.
still a lot of shared styles with history i
| |
| 95 </style> | |
| 96 | |
| 97 <div id="main-container"> | |
| 98 <div id="title" aria-expanded$="[[isExpanded(cardOpen)]]" | |
|
calamity
2016/02/02 04:09:47
Let's get rid of this isExpanded stuff.
yingran
2016/02/09 04:21:34
Done.
| |
| 99 aria-controls="collapse" on-tap="toggleTabCard"> | |
| 100 <div> | |
| 101 {{device}} | |
| 102 <span id="last-update-time">{{lastUpdateTime}}</span> | |
| 103 </div> | |
| 104 <iron-icon icon="expand-less" id="dropdown-indicator"></iron-icon> | |
| 105 </div> | |
| 106 | |
| 107 <iron-collapse opened="{{cardOpen}}" id="collapse"> | |
| 108 <div id="tab-item-list"> | |
| 109 <template is="dom-repeat" items="{{tabs}}" as="tab" id="tab-list"> | |
| 110 <div class="item-container"> | |
| 111 <div class="website-icon" id="icon"></div> | |
| 112 <a href="{{tab.url}}" id="tab-url">{{tab.title}}</a> | |
| 113 </div> | |
| 114 </template> | |
| 115 <div class="item-container"> | |
| 116 <p on-tap="openAllTabs_" id="open-tabs">Open all in new tab(s)</p> | |
|
calamity
2016/02/02 04:09:47
This gon need some i18n.
yingran
2016/02/09 04:21:34
Done.
| |
| 117 </div> | |
| 118 </div> | |
| 119 </iron-collapse> | |
| 120 | |
| 121 </div> | |
| 122 </template> | |
| 123 <script src="chrome://history/synced_device_card.js"></script> | |
| 124 </dom-module> | |
| OLD | NEW |