Skip to content

fix: adapt to AmberMpris changes compared to QtMpris (another atempt)#19

Closed
jmlich wants to merge 5 commits intopiggz:masterfrom
jmlich:interface-amber-mpris
Closed

fix: adapt to AmberMpris changes compared to QtMpris (another atempt)#19
jmlich wants to merge 5 commits intopiggz:masterfrom
jmlich:interface-amber-mpris

Conversation

@jmlich
Copy link
Copy Markdown
Contributor

@jmlich jmlich commented Mar 19, 2026

Built on top of #18
fixes showing of title, album, and artist - using pointer instead of copy
artist is actualy in contributingArtist field.
adding some qml interface for convinient testing.
fixes duration (removes /1000)

@jmlich jmlich changed the title fix: adapt to AmberMpris changes compared to QtMpris fix: adapt to AmberMpris changes compared to QtMpris (another atempt) Mar 19, 2026
@PureTryOut
Copy link
Copy Markdown
Contributor

I'd prefer it if you just review my PR and tell me what I need to change so I can fix it myself and we'll just get it in correctly directly there, rather than you making another branch which includes my changes plus some of your own stuff.

@jmlich
Copy link
Copy Markdown
Contributor Author

jmlich commented Mar 20, 2026

I am sorry, In some point testing turned to debugging.

@jmlich jmlich closed this Mar 20, 2026
@jmlich jmlich deleted the interface-amber-mpris branch March 21, 2026 07:19
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