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

Unified Diff: third_party/WebKit/Source/core/css/mediaControlsNew.css

Issue 1079323002: Support text track selection in video controls (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase and addressed fs' comments Created 4 years, 10 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/css/mediaControlsNew.css
diff --git a/third_party/WebKit/Source/core/css/mediaControlsNew.css b/third_party/WebKit/Source/core/css/mediaControlsNew.css
index 8b82dbf38019b6c79307403fbbbfe9c77a08c65f..f7d196667c10fab64fa9c691048e13f947f93caf 100644
--- a/third_party/WebKit/Source/core/css/mediaControlsNew.css
+++ b/third_party/WebKit/Source/core/css/mediaControlsNew.css
@@ -44,6 +44,7 @@ video::-webkit-media-controls {
direction: ltr;
display: flex;
flex-direction: column;
+ font-family: Segoe, "Helvetica Neue", Roboto, Arial, Helvetica, sans-serif ;
justify-content: flex-end;
align-items: center;
}
@@ -80,7 +81,6 @@ audio::-webkit-media-controls-panel, video::-webkit-media-controls-panel {
/* The duration is also specified in MediaControlElements.cpp and LayoutTests/media/media-controls.js */
transition: opacity 0.3s;
- font-family: Segoe, "Helvetica Neue", Roboto, Arial, Helvetica, sans-serif ;
font-size: 14px;
font-weight: normal; /* Make sure that we don't inherit non-defaults. */
font-style: normal;
@@ -368,6 +368,68 @@ audio::-webkit-media-controls-fullscreen-volume-max-button, video::-webkit-media
display: none;
}
+video::-internal-media-controls-text-track-list {
+ position: absolute;
+ bottom: 48px;
+ right: 5px;
+ background-color: #fafafa;
+ max-width: 50%;
+ max-height: 250px;
+ width: 150px;
+ overflow-x: hidden;
+ overflow-y: auto;
+ text-overflow: ellipsis;
+ margin-bottom: 5px;
+ white-space: nowrap;
+ font-size: 14px;
+ padding: 8px 0px;
+}
+
+video::-internal-media-controls-text-track-list-item {
+ display: block;
+ color: #424242;
+ text-align: start;
+ line-height: 40px;
+ padding-right: 16px;
+}
+
+video::-internal-media-controls-text-track-list-item:hover {
+ background-color: #e0e0e0;
+}
+
+video::-internal-media-controls-text-track-list-item-input {
+ -webkit-appearance: media-track-selection-checkmark;
+ position: relative;
+ visibility: hidden;
+ top: -2px;
+ left: 0;
+ vertical-align: middle;
+ margin: 0 5px 0 0;
+ width: 16px;
+ height: 16px;
+ margin-left: 12px;
+}
+
+video::-internal-media-controls-text-track-list-item-input:checked {
+ visibility: visible;
+}
+
+video::-internal-media-controls-text-track-list-kind-captions {
+ -webkit-appearance: media-closed-captions-icon;
+ height: 20px;
+ width: 20px;
+ margin-left: 10px;
+ vertical-align: middle;
+}
+
+video::-internal-media-controls-text-track-list-kind-subtitles {
+ -webkit-appearance: media-subtitles-icon;
+ height: 20px;
+ width: 20px;
+ margin-left: 10px;
+ vertical-align: middle;
+}
+
video::-webkit-media-text-track-container {
position: relative;
width: inherit;

Powered by Google App Engine
This is Rietveld 408576698