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

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

Issue 2382993003: Video player: Reset the focus on entering fullscree mode. (Closed)
Patch Set: 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/file_manager/video_player/js/media_controls.js
diff --git a/ui/file_manager/video_player/js/media_controls.js b/ui/file_manager/video_player/js/media_controls.js
index b718b12718298eebfed5a02ca5cf23e278d36e3a..a2b44eb160d9e17ab3ce523e4c8e10ae94bce0ca 100644
--- a/ui/file_manager/video_player/js/media_controls.js
+++ b/ui/file_manager/video_player/js/media_controls.js
@@ -1093,5 +1093,10 @@ VideoControls.prototype.onFullScreenChanged = function(fullscreen) {
this.fullscreenButton_.setAttribute('aria-label',
fullscreen ? str('VIDEO_PLAYER_EXIT_FULL_SCREEN_BUTTON_LABEL')
: str('VIDEO_PLAYER_FULL_SCREEN_BUTTON_LABEL'));;
+ // If the fullscreen button has focus on entering fullscreen mode, reset the
+ // focus to make the spacebar toggle play/pause state. This is the
+ // consistent behavior with Youtube Web UI.
+ if (fullscreen)
+ this.fullscreenButton_.blur();
}
};
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698