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

Side by Side Diff: Source/core/css/mediaControls.css

Issue 658353002: Remove background color from overlay cast button (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Fix Peter's nit on CSS, and add rendering layout tests. Created 6 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
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2009 Apple Inc. All rights reserved. 2 * Copyright (C) 2009 Apple Inc. All rights reserved.
3 * Copyright (C) 2009 Google Inc. 3 * Copyright (C) 2009 Google Inc.
4 * 4 *
5 * Redistribution and use in source and binary forms, with or without 5 * Redistribution and use in source and binary forms, with or without
6 * modification, are permitted provided that the following conditions 6 * modification, are permitted provided that the following conditions
7 * are met: 7 * are met:
8 * 1. Redistributions of source code must retain the above copyright 8 * 1. Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * 2. Redistributions in binary form must reproduce the above copyright 10 * 2. Redistributions in binary form must reproduce the above copyright
(...skipping 127 matching lines...) Expand 10 before | Expand all | Expand 10 after
138 video::-webkit-media-controls-overlay-play-button { 138 video::-webkit-media-controls-overlay-play-button {
139 -webkit-appearance: media-overlay-play-button; 139 -webkit-appearance: media-overlay-play-button;
140 display: flex; 140 display: flex;
141 position: absolute; 141 position: absolute;
142 top: 50%; 142 top: 50%;
143 left: 50%; 143 left: 50%;
144 margin-left: -40px; 144 margin-left: -40px;
145 margin-top: -40px; 145 margin-top: -40px;
146 border: none; 146 border: none;
147 box-sizing: border-box; 147 box-sizing: border-box;
148 background-color: transparent;
149 width: 80px; 148 width: 80px;
150 height: 80px; 149 height: 80px;
151 padding: 0; 150 padding: 0;
152 } 151 }
153 152
154 video::-internal-media-controls-overlay-cast-button { 153 video::-internal-media-controls-overlay-cast-button {
155 -webkit-appearance: -internal-media-overlay-cast-off-button; 154 -webkit-appearance: -internal-media-overlay-cast-off-button;
156 display: flex; 155 display: flex;
157 position: absolute; 156 position: absolute;
158 top: 5%; 157 top: 5%;
159 left: 5%; 158 left: 5%;
160 margin-left: 0px; 159 margin-left: 0px;
161 margin-top: 0px; 160 margin-top: 0px;
162 border: none; 161 border: none;
163 box-sizing: border-box; 162 box-sizing: border-box;
164 background-color: rgba(0,0,0,0.5); 163 color: inherit;
Peter Beverloo 2014/10/20 19:08:48 Sorry, I meant remove the "color" property, not th
aberent 2014/10/20 20:13:50 Done.
165 width: 30px; 164 width: 30px;
166 height: 30px; 165 height: 30px;
167 padding: 0; 166 padding: 0;
168 } 167 }
169 168
170 audio::-webkit-media-controls-play-button, video::-webkit-media-controls-play-bu tton { 169 audio::-webkit-media-controls-play-button, video::-webkit-media-controls-play-bu tton {
171 -webkit-appearance: media-play-button; 170 -webkit-appearance: media-play-button;
172 display: flex; 171 display: flex;
173 flex: none; 172 flex: none;
174 border: none; 173 border: none;
(...skipping 48 matching lines...) Expand 10 before | Expand all | Expand 10 after
223 text-decoration: none; 222 text-decoration: none;
224 } 223 }
225 224
226 audio::-webkit-media-controls-timeline, video::-webkit-media-controls-timeline { 225 audio::-webkit-media-controls-timeline, video::-webkit-media-controls-timeline {
227 -webkit-appearance: media-slider; 226 -webkit-appearance: media-slider;
228 display: flex; 227 display: flex;
229 flex: 1 1 auto; 228 flex: 1 1 auto;
230 height: 8px; 229 height: 8px;
231 margin: 0 15px 0 0; 230 margin: 0 15px 0 0;
232 padding: 0; 231 padding: 0;
233 background-color: transparent;
234 min-width: 25px; 232 min-width: 25px;
235 border: initial; 233 border: initial;
236 color: inherit; 234 color: inherit;
237 } 235 }
238 236
239 audio::-webkit-media-controls-volume-slider, video::-webkit-media-controls-volum e-slider { 237 audio::-webkit-media-controls-volume-slider, video::-webkit-media-controls-volum e-slider {
240 -webkit-appearance: media-volume-slider; 238 -webkit-appearance: media-volume-slider;
241 display: flex; 239 display: flex;
242 /* The 1.9 value was empirically chosen to match old-flexbox behaviour 240 /* The 1.9 value was empirically chosen to match old-flexbox behaviour
243 * and be aesthetically pleasing. 241 * and be aesthetically pleasing.
244 */ 242 */
245 flex: 1 1.9 auto; 243 flex: 1 1.9 auto;
246 height: 8px; 244 height: 8px;
247 max-width: 70px; 245 max-width: 70px;
248 margin: 0 15px 0 0; 246 margin: 0 15px 0 0;
249 padding: 0; 247 padding: 0;
250 background-color: transparent;
251 min-width: 15px; 248 min-width: 15px;
252 border: initial; 249 border: initial;
253 color: inherit; 250 color: inherit;
254 } 251 }
255 252
256 /* FIXME these shouldn't use special pseudoShadowIds, but nicer rules. 253 /* FIXME these shouldn't use special pseudoShadowIds, but nicer rules.
257 https://code.google.com/p/chromium/issues/detail?id=112508 254 https://code.google.com/p/chromium/issues/detail?id=112508
258 https://bugs.webkit.org/show_bug.cgi?id=62218 255 https://bugs.webkit.org/show_bug.cgi?id=62218
259 */ 256 */
260 input[type="range" i]::-webkit-media-slider-container { 257 input[type="range" i]::-webkit-media-slider-container {
(...skipping 151 matching lines...) Expand 10 before | Expand all | Expand 10 after
412 font-weight: bold; 409 font-weight: bold;
413 } 410 }
414 411
415 video::-webkit-media-text-track-container u { 412 video::-webkit-media-text-track-container u {
416 text-decoration: underline; 413 text-decoration: underline;
417 } 414 }
418 415
419 video::-webkit-media-text-track-container i { 416 video::-webkit-media-text-track-container i {
420 font-style: italic; 417 font-style: italic;
421 } 418 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698