Skip to content

Conversation

@fem-al-e
Copy link

Changes made

General

  • updated the header warning to reflect mercurial's promotion to community moderator

Media Player List

  • added support for mpv and Yandex Music
  • general writing improvements to the key of the media player table
  • numerically sorted media player list annotations
  • unified writing style for media player list annotations
  • added annotation for Yandex Music

Additional Notes

  • added general explanation for usage of SMTC & MPRIS for clarity
  • unified writing style for all "helper program required" notes

Players with known issues

  • reduced overall explanation length

Comment

As mentioned on Discord, this would probably fare better as a GitHub wiki in the long term...
I might look into it this weekend.

comment: this would probably fare better as a github wiki in the long term...
Copy link
Owner

@ungive ungive left a comment

Choose a reason for hiding this comment

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

Thank you for your PR! I've left some notes that should be addressed.

In my opinion working on this document is not really worth the time, you should instead focus on the wiki that you talked about in the Discord. I'm still happy to merge it, since you've put the work in. Thank you for that!

A tip for you for future PRs: Try to change only what needs changes, as the maintainer has to go through everything that you updated and that process takes time. For some changes the review can take significantly more time than actually making the change. I appreciate your effort, but e.g. reordering the footnotes in the table means that I have to go through all of them to check if they are still correct, for no apparent gain. (I didn't check those, the table should just be removed)

I encourage you to work on the wiki, then I'll add it to the music-presence GitHub org and we can just link to it and the player support website in this document. 😄

Comment on lines +4 to +5
> **Only the ["Additional Notes"](#additional-notes) and ["Players with known issues"](#players-with-known-issues) sections are actively maintained.**
> An accurate list of media players is available [here](https://musicpresence.pocha.moe/) - thank you to our community moderator [@mercurialworld](https://github.com/mercurialworld) for making this site!
Copy link
Owner

Choose a reason for hiding this comment

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

It probably makes more sense to link to the wiki in various places once it exists and label it maintained, when there is someone to actually maintain it. I don't think maintaining this document via PRs is the way to go.

Copy link
Owner

Choose a reason for hiding this comment

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

So I wouldn't call this actively maintained here.

## Additional notes

### MusicBee
---
Copy link
Owner

Choose a reason for hiding this comment

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

I think this table should just be removed entirely instead of putting any work in to update it.

Comment on lines +107 to +111
Windows and Linux offer specific tools for media players to report their playback information, named SMTC and MPRIS respectively.
These are the essential features that allow Music Presence to function on Windows and Linux.

While foobar2000 reports basic information out of the box,
this plugin will add the album name and a progress bar to your status:
https://github.com/ungive/foo_mediacontrol
Unfortunately, some players do not utilize this correctly, while others report nothing at all.
For these cases, various people have written helper software to "close the gap".
Copy link
Owner

Choose a reason for hiding this comment

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

While nice, I think most people who visit here just want to know what they need to download. 😅

### MusicBee

### AIMP
MusicBee requires [this plugin](https://github.com/HenryPDT/mb_MediaControl) to work.
Copy link
Owner

Choose a reason for hiding this comment

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

Maybe noting some of the quirks of the plugin might make sense, so people know what to expect. I think there were issues with the song sometimes not updating, missing or delayed cover images and badly functioning media keys. Can't say with 100% certainty though.

### AIMP

AIMP requires [this plugin](https://www.aimp.ru/forum/index.php?topic=63341) to work.
While it generally works fine, it does apparently provide wrong playback information at times.
Copy link
Owner

Choose a reason for hiding this comment

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

Why "at times"? It never does it, iirc. The use of the word "apparently" is superfluous imo.

I think the original description of its quirks were fine.

Comment on lines +129 to +130
VLC requires [this plugin](https://github.com/spmn/vlc-win10smtc) to work.
As of right now, this plugin reports neither cover images, nor an accurate playback position.
Copy link
Owner

Choose a reason for hiding this comment

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

Only keep the first line here.

Thumbnail reporting was added here: spmn/vlc-win10smtc@69b69c0

I also added playback position reporting in this PR: spmn/vlc-win10smtc#7

Comment on lines +196 to +198
Some players report their playback position with an outdated “as of” timestamp.
This will show up as an elapsed timer that’s wildly ahead of reality (e.g., “13:04 elapsed” right after resuming a 3-minute track).
It’s most noticeable after pausing and later resuming: Discord keeps the stale timestamp, so the counter jumps.
Copy link
Owner

Choose a reason for hiding this comment

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

Use straight quotation marks: " and '

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants