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

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: added more tests 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 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..20b14b6efeacaa394e7d88f36afb54f151e3f02b 100644
--- a/third_party/WebKit/Source/core/css/mediaControlsNew.css
+++ b/third_party/WebKit/Source/core/css/mediaControlsNew.css
@@ -418,6 +418,47 @@ video::-internal-media-controls-text-track-list-kind-subtitles {
vertical-align: middle;
}
+video::-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 {
mlamouri (slow - plz ping) 2016/09/01 18:29:15 Can you merge this with the closed captions one in
kdsilva 2016/09/05 14:39:35 Done.
+ position: absolute;
+ bottom: 48px;
+ right: 0px;
+ background-color: #fafafa;
+ max-width: 50%;
+ max-height: 250px;
+ min-width: 150px;
+ overflow-x: hidden;
+ overflow-y: auto;
+ white-space: nowrap;
+ font-size: 14px;
+ padding: 8px 0px;
+}
+
+video::-internal-media-controls-overflow-menu-list-item {
+ display: block;
+ color: #424242;
+ line-height: 40px;
+ padding-left: 28px;
+}
+
+video::-internal-media-controls-overflow-menu-list-item:hover {
+ background-color: #e0e0e0;
+}
+
video::-webkit-media-text-track-container {
position: relative;
width: inherit;

Powered by Google App Engine
This is Rietveld 408576698