huynhsontung / Screenbox

LibVLC-based media player for the Universal Windows Platform
GNU General Public License v3.0
957 stars 37 forks source link

Add informations about codecs #47

Open ChabaneAmaury opened 1 year ago

ChabaneAmaury commented 1 year ago

Just as in the title.

Could you please add all the informations on the codec used on the played file ? Just like in VLC

PS: what do you mean exactly by "VLC based media player"?

huynhsontung commented 1 year ago

Right now you can right click -> Properties to view some basic info on the media file. This info is limited by the Windows API at the moment so not every file will have all the info available, or as detailed as VLC media info. I haven't looked into VLC media info query but I think it would be better. Needs more investigation.

Screenbox uses LibVLCSharp, which uses native libvlc DLL under the hood, to perform media decoding and playback. Hence "VLC based media player".

realgooseman commented 1 month ago

Hi @huynhsontung, any updates on this ?

Right click -> Properties doesn't even show video resolution most of the time. What does VLC use for those file informations that Screenbox lacks ?

There's this tool called MediaInfo which adds a context menu entry to show detailed file information, is there any way to include what this tool does to Screenbox ? or something with ffmpeg,... to extract video information ?

huynhsontung commented 1 month ago

Hi @realgooseman. Screenbox currently doesn't get media info from VLC but relies mostly on Windows APIs, which are limited by the codecs Windows supports. There is no technical reason that prevents Screenbox from getting more accurate information using VLC. I just haven't gotten around to implementing it yet. Is this feature critical for your use case?

realgooseman commented 1 month ago

I recently ditched VLC for Screenbox as it's more modern and straightforward, plus it uses a VideoLan library so performance should be similar to VLC but video information is one of the first things I noticed that was missing on Screenbox. I wouldn't say it's critical for my use case but it's a very useful feature and it will probably be asked by advanced users when Screenbox gains popularity.

Anyway, great job with Screenbox, I look forward to the project's progress.