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

Unified 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: rebased 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 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 1011a2ab91fee8afcf2d1a59f4d39daf3ceb4b9e..348ac8508ee2da33144ad6ec4b02e2c5170e96e8 100644
--- a/third_party/WebKit/Source/core/css/mediaControlsNew.css
+++ b/third_party/WebKit/Source/core/css/mediaControlsNew.css
@@ -359,7 +359,7 @@ audio::-webkit-media-controls-fullscreen-volume-max-button, video::-webkit-media
display: none;
}
-video::-internal-media-controls-text-track-list {
+video::-internal-media-controls-text-track-list, video::-internal-media-controls-overflow-menu-list, audio::-internal-media-controls-overflow-menu-list {
position: absolute;
bottom: 48px;
right: 0px;
@@ -383,7 +383,7 @@ video::-internal-media-controls-text-track-list-item {
text-overflow: ellipsis;
}
-video::-internal-media-controls-text-track-list-item:hover {
+video::-internal-media-controls-text-track-list-item:hover, video::-internal-media-controls-overflow-menu-list-item:hover, audio::-internal-media-controls-overflow-menu-list-item:hover {
background-color: #e0e0e0;
}
@@ -418,6 +418,28 @@ video::-internal-media-controls-text-track-list-kind-subtitles {
vertical-align: middle;
}
+video::-internal-overflow-menu-button, audio::-internal-overflow-menu-button {
+ -webkit-appearance: -internal-overflow-menu-button;
+ display: flex;
+ flex: none;
+ box-sizing: border-box;
+ width: 32px;
+ height: 32px;
+ padding: 0px;
+ border-width: 0px;
+ margin-left: 0px;
+ margin-right: 0px;
+ background-color: initial;
+ color: inherit;
+}
+
+video::-internal-media-controls-overflow-menu-list-item, audio::-internal-media-controls-overflow-menu-list-item {
+ display: block;
+ color: #424242;
+ line-height: 40px;
+ padding-left: 28px;
+}
+
video::-webkit-media-text-track-container {
position: relative;
width: inherit;

Powered by Google App Engine
This is Rietveld 408576698