Modernflyouts: Add support for Dopamine music player for Windows

Created on 18 Sep 2020  路  16Comments  路  Source: ModernFlyouts-Community/ModernFlyouts

Please add support for stop, repeat, shuffle, timeline info for the app Dopamine music player.

Link - https://github.com/digimezzo/dopamine-windows

Enhancement Wontfix

Most helpful comment

@Cyberdroid1, I have also added timeline info support to it in https://github.com/digimezzo/dopamine-windows/pull/1206

Let's hope this will also get merged

All 16 comments

Is that supported by the stock windows flyout?

Stock flyout just shows play/pause, previous, next. Same supported by ModernFlyouts. Don't know whether it supports skip, repeat, stop, etc.

It's up-to the app whether to support repeat, stop, shuffle & etc. We're just using a public API that has some limitations. I would recommend you to open a issue on the dopamine-windows repo and ask them to support these controls. We can't do anything about this.

If the repo is active and I have free time, I can open a pull request with the said feature.

The repo is active. I will open the issue.

A weird bug now. Whenever I launch this app and use shortcuts to skip, play/pause, etc. in this music player, the system UI pops up with the media flyout. ModernFlyout is not even showing the media flyout! It doesnt know whether music is playing or not.

Would request to reopen this issue.

Yep reopen it. Seems to be a different issue. But it may take weeks to fix this one.

I don't have the button to reopen it, weirdly.

image

Done!

Thanks. I really think I am cursed sometimes because of the kinds of bugs I face. XD

Can you try reproducing this issue on your side? I expect the same behaviour as mine.

Hey @Cyberdroid1

Is this what you are saying?

image

image

Currently they doesn't support shuffle/repeat/stop externally, so we can't hook those controls.

I contacted @digimezzo, they are open for contributions. I will open a PR for this.

Yes, I meant this only.

Thanks so much!

@Cyberdroid1,

Done! Opened a PR on the target. I will close this issue, you can track the PR on that repo. If it gets merged, it will be good for us.

The PR : https://github.com/digimezzo/dopamine-windows/pull/1201

My part's done.

Hey @Cyberdroid1, the PR has been accepted & got merged by the Dopamine-windows team 馃帄馃帄馃帄馃帀馃帀馃帀

Yay! Thanks so much, bro! Nice to see this get accepted!

@Cyberdroid1, I have also added timeline info support to it in https://github.com/digimezzo/dopamine-windows/pull/1206

Let's hope this will also get merged

Was this page helpful?
0 / 5 - 0 ratings