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

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

Issue 1441603002: Revert of Fix accessibility issues in AudioPlayer and VideoPlayer. (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
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 f0e3b9d325901bf2fc55bcb88e4e610e05e01dd1..f43dfba10f5dc89a4d3c98f873dac9a976618f9b 100644
--- a/ui/file_manager/video_player/css/media_controls.css
+++ b/ui/file_manager/video_player/css/media_controls.css
@@ -11,6 +11,14 @@
height: 32px;
position: relative;
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 {
@@ -134,15 +142,6 @@
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 {
@@ -155,10 +154,6 @@
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. */
@@ -168,7 +163,7 @@
url(../images/media/2x/media_fullscreen.png) 2x);
}
-#controls[fullscreen] .media-button.fullscreen {
+#video-player[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/integration_tests/file_manager/open_audio_files.js ('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