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

Unified Diff: ui/file_manager/video_player/css/media_controls.css

Issue 1435253002: VideoPlayer: Fix accessibility issues. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 1 month 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
« no previous file with comments | « ui/file_manager/file_manager_resources.grd ('k') | ui/file_manager/video_player/js/media_controls.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/file_manager/video_player/css/media_controls.css
diff --git a/ui/file_manager/video_player/css/media_controls.css b/ui/file_manager/video_player/css/media_controls.css
index f43dfba10f5dc89a4d3c98f873dac9a976618f9b..f0e3b9d325901bf2fc55bcb88e4e610e05e01dd1 100644
--- a/ui/file_manager/video_player/css/media_controls.css
+++ b/ui/file_manager/video_player/css/media_controls.css
@@ -13,14 +13,6 @@
width: 32px;
}
-.media-button:hover {
- background-color: rgba(153, 153, 153, 0.2);
-}
-
-.media-button:active {
- background-color: rgba(153, 153, 153, 0.4);
-}
-
.media-button.disabled {
background-color: transparent;
opacity: 0.26;
@@ -142,6 +134,15 @@ paper-slider.volume {
url(../images/media/media_chromecast.png) 1x,
url(../images/media/2x/media_chromecast.png) 2x);
display: none;
+ border-radius: 2px;
+}
+
+/* Reset browser's button style. */
+.media-button.cast {
+ background-color: transparent;
+ border: none;
+ cursor: pointer;
+ outline: none;
}
#video-player[cast-available][castable] .media-button.cast {
@@ -154,6 +155,10 @@ paper-slider.volume {
url(../images/media/2x/media_chromecast_casting.png) 2x);
}
+.media-button.cast:focus:not(.using-mouse) {
+ box-shadow: 0 0 0 1px rgba(66, 133, 244, 0.5);
+}
+
/* Fullscreen button. */
/* There is no final decision whether we need a separate icon when toggled. */
@@ -163,7 +168,7 @@ paper-slider.volume {
url(../images/media/2x/media_fullscreen.png) 2x);
}
-#video-player[fullscreen] .media-button.fullscreen {
+#controls[fullscreen] .media-button.fullscreen {
background-image: -webkit-image-set(
url(../images/media/media_fullscreen_exit.png) 1x,
url(../images/media/2x/media_fullscreen_exit.png) 2x);
« no previous file with comments | « ui/file_manager/file_manager_resources.grd ('k') | ui/file_manager/video_player/js/media_controls.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698