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

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: Addressed comments Created 4 years, 3 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::-internal-overflow-menu-button {
kdsilva 2016/08/29 20:44:13 I followed the way the closed captions button was
liberato (no reviews please) 2016/08/30 16:42:06 i think that most of the entries are of the form
422 -webkit-appearance: -internal-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-hidden {
459 display:none;
460 }
461
462 video::-internal-media-controls-overflow-menu-list-item-element {
463 display: block;
464 color: #424242;
465 text-align: start;
466 line-height: 40px;
467 padding-right: 16px;
468 text-overflow: ellipsis;
469 display: none;
470 }
471 video::-internal-media-controls-overflow-menu-list-item:hover {
472 background-color: #e0e0e0;
473 }
474
421 video::-webkit-media-text-track-container { 475 video::-webkit-media-text-track-container {
422 position: relative; 476 position: relative;
423 width: inherit; 477 width: inherit;
424 height: inherit; 478 height: inherit;
425 overflow: hidden; 479 overflow: hidden;
426 480
427 font: 22px sans-serif; 481 font: 22px sans-serif;
428 text-align: center; 482 text-align: center;
429 color: rgba(255, 255, 255, 1); 483 color: rgba(255, 255, 255, 1);
430 484
(...skipping 54 matching lines...) Expand 10 before | Expand all | Expand 10 after
485 font-weight: bold; 539 font-weight: bold;
486 } 540 }
487 541
488 video::cue(u) { 542 video::cue(u) {
489 text-decoration: underline; 543 text-decoration: underline;
490 } 544 }
491 545
492 video::cue(i) { 546 video::cue(i) {
493 font-style: italic; 547 font-style: italic;
494 } 548 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698