D21554: Allow setting fullscreen via MPRIS
Kai Uwe Broulik
noreply at phabricator.kde.org
Mon Jun 3 08:15:14 BST 2019
broulik created this revision.
broulik added reviewers: Plasma, fvogt.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.
REVISION SUMMARY
This implements `CanSetFullscreen` to return `true` when it's a video and lets the `Fullscreen` property reflect and change the fullscreen state of the player.
BUG: 408059
TEST PLAN
YouTube sets the entire website as fullscreen and gets confused when just fullscreening the player but it works fine in e.g. Vimeo.
If the video autostarted it refuses to fullscreen it as there hasn't benn any user interaction, otherwise works fine.
REPOSITORY
R856 Plasma Browser Integration
REVISION DETAIL
https://phabricator.kde.org/D21554
AFFECTED FILES
extension/content-script.js
extension/extension.js
host/mprisplugin.cpp
host/mprisplugin.h
To: broulik, #plasma, fvogt
Cc: plasma-devel, ericadams, jraleigh, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190603/0fceb776/attachment.html>
More information about the Plasma-devel
mailing list