This looks really good! Great work! There's a few things that I'd recommend changing, they are mostly alignment issues.

Line up the top of the image with the Play Music icon (currently it's centered with the Tracklist, but the user won't expand this often enough to look proper) , this will also allow the author to be on a singular line, rather than taking up three lines.
I'd also move the author line to the bottom left instead of the bottom right.

The image is missing it's outline as seen in for example the object selector. This has to be re-added to allign with ui's standard. (It currently pops out, but it has to be inside a frame: the bottom is lighter and the top is darker).

I will make sure to playtest this right away and will leave another comment afterwards.

I made an edit of the window using photoshop of what the final result should look like.

I noticed while play testing that the image is dynamic within the window, but this one can be kept static in its y value.

By moving the Author to the bottom left rather than the bottom right it allows for the window to be slightly smaller, too and have less empty space.

UI is nice and responsive. No changes necessary there.

Hope the feedback helps! :)

Suggested changes above.

Just updated the code. It looks much cleaner now. Any more feedback is appreciated! :)

@karst do we want to add some string like "Composers" or "Performers" or something?

I don't think that's necessary. Most songs seperate artist from song using just a dash, which makes that clear enough.

Perhaps we should update the music objects to include Allister Brimble where necessary.

Would it be possible to hide the scroll bars from the track list if there's no need to scroll?

Right now this tab seems very cluttered. 🥴

I was thinking,
I requested if all audio objects could have their composers listed. This has since been done.
I think it's no longer necessary to list the object author(s). This would clear up another line.

Would it be possible to hide the scroll bars from the track list if there's no need to scroll?

Right now this tab seems very cluttered. 🥴

Working on this now.

This pull request has been marked as stale and will be closed in 14 days if no action is taken. To keep it open, leave a comment or remove the stale-pr label. If you're awaiting feedback from a developer, please send us a reminder (either here or on Discord).

Going over stale PRs, I came across this one. To be honest, I think this adds a lot of clutter to the UI. Thinking about it, I believe this has to do with seeing boldface black text on a red window. Perhaps it would look better using the 'Small' font style, rather than the default 'Medium' one? @karst, what do you think?

Before:

After:

Aside from the UI, I think the code needs a bit of cleaning up before this is ready to be merged. Considering how long this PR has been sitting for, I'm willing to take it to the finish line myself if need be, though.

The smaller text, which you've shown in the screenshot definitely looks cleaner.

The author on the bottom no longer needs to be shown there anymore since each song has it's componist listed since a few months now.

Photoshop

After some discussion with @Gymnasiast on Discord, we agreed it would be best to push the preview image down a little, such that it aligned horizontally with the track list. This would provide ample room for a future button next to the music dropdown above.

Here's a few screenshots with the current status as the rewrites I've pushed.

Music off:

Music on:

Very nice, perhaps push the full window down by one or two more pixels so the resize icon doesn't overlap with other buttons?

I noticed one oddity when testing. If you click on another window the sideways scrollbar disappears.

I noticed one oddity when testing. If you click on another window the sideways scrollbar disappears.

Urgh, this was due a race condition inherent to the widget definitions being shared globally. I've added a hack to counter this.

@karst If you're happy with the PR in its current state, I think it's ready to be merged. Just needs a changelog entry and we're good to go.