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