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

Side by Side Diff: third_party/WebKit/Source/core/css/mediaControlsNew.css

Issue 2243473002: Adding overflow menu to media player (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 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) 2015 Google Inc. 3 * Copyright (C) 2015 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 400 matching lines...) Expand 10 before | Expand all | Expand 10 after
411 } 411 }
412 412
413 video::-internal-media-controls-text-track-list-kind-subtitles { 413 video::-internal-media-controls-text-track-list-kind-subtitles {
414 -webkit-appearance: -internal-media-subtitles-icon; 414 -webkit-appearance: -internal-media-subtitles-icon;
415 height: 20px; 415 height: 20px;
416 width: 20px; 416 width: 20px;
417 margin-left: 10px; 417 margin-left: 10px;
418 vertical-align: middle; 418 vertical-align: middle;
419 } 419 }
420 420
421 video::-webkit-overflow-menu-button {
whywhat 2016/08/19 23:56:55 ditto wrt internal we should not add webkit prefix
kdsilva 2016/08/24 05:33:25 Would you recommend changing the name to Blink? Pa
Timothy Loh 2016/08/24 06:19:38 We are not adding new -webkit features (or -blink)
kdsilva 2016/08/26 18:11:02 Got it, that makes sense. Thanks for the details!
422 -webkit-appearance: media-overflow-menu-button;
423 display: flex;
424 flex: none;
425 box-sizing: border-box;
426 width: 32px;
427 height: 32px;
428 padding: 0px;
429 border-width: 0px;
430 margin-left: 0px;
431 margin-right: 0px;
432 background-color: initial;
433 color: inherit;
434 }
435
436 video::-internal-media-controls-overflow-menu-list {
437 position: absolute;
438 bottom: 48px;
439 right: 0px;
440 background-color: #fafafa;
441 max-width: 50%;
442 max-height: 250px;
443 min-width: 150px;
444 overflow-x: hidden;
445 overflow-y: auto;
446 white-space: nowrap;
447 font-size: 14px;
448 padding: 8px 0px;
449 }
450
451 video::-internal-media-controls-overflow-menu-list-item {
452 display: block;
453 color: #424242;
454 line-height: 40px;
455 padding-left: 28px;
456 }
457
458 video::-internal-media-controls-overflow-menu-list-item-element {
459 display: block;
460 color: #424242;
461 text-align: start;
462 line-height: 40px;
463 padding-right: 16px;
464 text-overflow: ellipsis;
465 display: none;
466 }
467 video::-internal-media-controls-overflow-menu-list-item:hover {
468 background-color: #e0e0e0;
469 }
470
421 video::-webkit-media-text-track-container { 471 video::-webkit-media-text-track-container {
422 position: relative; 472 position: relative;
423 width: inherit; 473 width: inherit;
424 height: inherit; 474 height: inherit;
425 overflow: hidden; 475 overflow: hidden;
426 476
427 font: 22px sans-serif; 477 font: 22px sans-serif;
428 text-align: center; 478 text-align: center;
429 color: rgba(255, 255, 255, 1); 479 color: rgba(255, 255, 255, 1);
430 480
(...skipping 54 matching lines...) Expand 10 before | Expand all | Expand 10 after
485 font-weight: bold; 535 font-weight: bold;
486 } 536 }
487 537
488 video::cue(u) { 538 video::cue(u) {
489 text-decoration: underline; 539 text-decoration: underline;
490 } 540 }
491 541
492 video::cue(i) { 542 video::cue(i) {
493 font-style: italic; 543 font-style: italic;
494 } 544 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698