Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New mpris #225

Merged
merged 5 commits into from
Mar 7, 2019
Merged

New mpris #225

merged 5 commits into from
Mar 7, 2019

Conversation

fcastilloec
Copy link
Member

This will update MPRIS to the new version which uses the new dbus-next module. Now we can use the same module for registering media keys.
The new module doesn't need the dbus dependencies for Linux making easier to build. We can now even build a 32bit version from a 64bit machine.
It also fixes my workaround for some properties of MPRIS where I had to add 1e-15

Latest version of mpris-service fixed the need to use 1e-15 on some variables
We have also added support for shuffle, repeat, position and seek
MediaKeys code only works for Mate and older version of gnome
All OSs using gnome3 use MPRIS to send mediakey shortcuts
@fcastilloec
Copy link
Member Author

I've tested in MATE, Gnome, Cinnamon, Gnome3, and Budgie; media keys and all mpris features are working on all of them.

@fcastilloec fcastilloec changed the title WIP: New mpris New mpris Mar 7, 2019
@fcastilloec
Copy link
Member Author

Also, this PR works together with the PR for the core, some of the new MPRIS features will only work with the new core version

@fcastilloec fcastilloec merged commit 3d11862 into master Mar 7, 2019
@fcastilloec fcastilloec deleted the new-mpris branch March 7, 2019 02:42
@acrisci
Copy link

acrisci commented Mar 17, 2019

👍 tag me in issues if you need help with the mpris-service stuff in the future.

@@ -96,36 +98,76 @@ module.exports = (win, player, app) => {
}
});

mprisPlayer.on('seek', (seek) => {
if (mprisPlayer.playbackStatus !== 'Stopped') {
if (seek < 0) seek = ~seek;
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@acrisci this is the only issue we have with mpris-service right now. I've already reported it here: dbusjs/mpris-service#27
We're waiting for the fix to come so I can remove this workaround and probably ship the new version

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just waiting on people to approve some other implementations before the release comes with this fix. Should be ready sometime in the next week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants