Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Delta Between Two Patch Sets: skin/popup.css

Issue 29317001: Relocated icon and redesigned icon popup (Closed)
Left Patch Set: Applied Sebastian's suggestions and updated strings Created Nov. 26, 2013, 4:18 p.m.
Right Patch Set: Merged setBadgeNumber and setBadgeBackgroundColor Created Dec. 13, 2013, 10:36 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« lib/stats.js ('K') | « safari/background.js ('k') | skin/popup.png » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 body 1 body
2 { 2 {
3 min-width: 250px; 3 min-width: 250px;
4 margin: 0px; 4 margin: 0px;
5 font-family: Segoe UI, Arial, sans-serif; 5 font-family: Segoe UI, Arial, sans-serif;
6 font-size: 13px; 6 font-size: 13px;
7 color: #545454; 7 color: #545454;
8 background-color: #f8f6f2; 8 background-color: #f8f6f2;
9 background-image: url(background.png); 9 background-image: url(background.png);
10 } 10 }
(...skipping 98 matching lines...) Expand 10 before | Expand all | Expand 10 after
109 footer 109 footer
110 { 110 {
111 cursor: pointer; 111 cursor: pointer;
112 padding: 10px 35px; 112 padding: 10px 35px;
113 border-top: 1px solid #e1ddd8; 113 border-top: 1px solid #e1ddd8;
114 background-image: url(background-main.png); 114 background-image: url(background-main.png);
115 } 115 }
116 116
117 footer:hover 117 footer:hover
118 { 118 {
119 background-image: url(background-main-hover.png); 119 background: -moz-linear-gradient(top, rgba(70, 50, 0, 0.1), rgba(70, 50, 0, 0. 1)),
120 url(background-main.png);
121 background: -webkit-linear-gradient(top, rgba(70, 50, 0, 0.1), rgba(70, 50, 0, 0.1)),
122 url(background-main.png);
123 background: linear-gradient(top, rgba(70, 50, 0, 0.1), rgba(70, 50, 0, 0.1)),
Wladimir Palant 2013/12/17 14:11:14 The standardized linear gradient syntax is actuall
124 url(background-main.png);
120 } 125 }
121 126
122 footer .icon 127 footer .icon
123 { 128 {
124 background-position: -50px -65px; 129 background-position: -50px -65px;
125 } 130 }
126 131
127 ul 132 ul
128 { 133 {
129 margin: 0px; 134 margin: 0px;
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
162 .icon 167 .icon
163 { 168 {
164 display: inline-block; 169 display: inline-block;
165 width: 15px; 170 width: 15px;
166 height: 15px; 171 height: 15px;
167 margin-right: 15px; 172 margin-right: 15px;
168 vertical-align: top; 173 vertical-align: top;
169 background-image: url(popup.png); 174 background-image: url(popup.png);
170 } 175 }
171 176
172 #enabled-on strong 177 #enabled-yes strong
173 { 178 {
174 font-weight: normal; 179 font-weight: normal;
175 color: #12610e; 180 color: #12610e;
176 } 181 }
177 182
178 #enabled-off strong 183 #enabled-no strong
179 { 184 {
180 font-weight: normal; 185 font-weight: normal;
181 color: #ac0505; 186 color: #ac0505;
182 } 187 }
183 188
184 #enabled .icon 189 #enabled .icon
185 { 190 {
186 background-position: -50px -1px ; 191 background-position: -50px -1px ;
187 } 192 }
188 193
189 #enabled.off .icon 194 #enabled.off .icon
Felix Dahlke 2013/12/02 15:45:58 I find it confusing that there's both #enabled-off
Thomas Greiner 2013/12/03 12:06:05 Done.
190 { 195 {
191 background-position: -50px -17px; 196 background-position: -50px -17px;
192 } 197 }
193 198
194 #enabled-off 199 #enabled-no,
195 { 200 #enabled.off #enabled-yes
196 display: none; 201 {
197 } 202 display: none;
198 203 }
199 #enabled.off #enabled-on 204
Felix Dahlke 2013/12/02 15:45:58 Can you merge this and the rule set above?
Thomas Greiner 2013/12/03 12:06:05 Done.
200 { 205 #enabled.off #enabled-no
201 display: none;
202 }
203
204 #enabled.off #enabled-off
205 { 206 {
206 display: inline; 207 display: inline;
207 } 208 }
208 209
209 #clickhide .icon, 210 #clickhide .icon,
210 #clickhide-cancel .icon 211 #clickhide-cancel .icon
211 { 212 {
212 background-position: -50px -33px; 213 background-position: -50px -33px;
213 } 214 }
214 215
(...skipping 108 matching lines...) Expand 10 before | Expand all | Expand 10 after
323 324
324 .checkbox[aria-checked="false"]:hover 325 .checkbox[aria-checked="false"]:hover
325 { 326 {
326 background-position: -34px -49px; 327 background-position: -34px -49px;
327 } 328 }
328 329
329 label 330 label
330 { 331 {
331 vertical-align: middle; 332 vertical-align: middle;
332 } 333 }
LEFTRIGHT

Powered by Google App Engine
This is Rietveld