Love the PDF reader! But the audio player seems kind of out-of-place. Could we perhaps style it to fit in a bit more?

No problem! I can change it

I've been doing some testing, and ogg files trigger the video player, not the audio player. Not sure if this is an issue at all.

Not ideal, but no blocking bug either, I'd say.

Agreed, especialy since that happens because lstat reports the type as video.

I made an attempt at creating controls for the audio player that fit more with the theme. I don't really like the way the volume controls look, but I haven't found a way to get any kind of speaker symbol that would respect the font colour of the user theme.

Woah, those look great! Any chance you two could coordinate on merging those changes? Otherwise, we'd go the two-PRs route.

That should be possible. @ghost-in-the-machine-86 how would it be best to get these changes to you? Or were you already working on something similar?

I did this media player, but i prefer your style @KjartanOli! Maybe i can add volume/progress bars to your player

Sounds great. I had no idea how to implement those. How should I send my code to you?

Idk how can you change my code from here (my first pull request), so i invited you to my repository

I just pushed my code to you develop branch.

thanks!

You'll have to merge develop with master in the fork for changes to appear in this PR!

Fixed ogg files to trigger audio player and applied @KjartanOli style to media player with some changes

Is this ready for review?

Yes