feat: hide redundant caption for videos when it only adds file extension

[Copilot]

For videos without metadata, the title displays the file's DisplayName (e.g., "MyVideo") while the caption shows the full filename (e.g., "MyVideo.mp4"), creating redundant information.

Result

• Before: Title: "MyVideo", Caption: "MyVideo.mp4"
• After: Title: "MyVideo.mp4", Caption: hidden

[Copilot]

Pull request overview

This PR improves the user experience for videos without metadata by eliminating redundant caption information. Previously, when a video file had no metadata, the title would display the file's DisplayName (e.g., "MyVideo") while the caption showed the full filename (e.g., "MyVideo.mp4"), creating unnecessary duplication. The PR changes this behavior to display only the full filename as the title when the caption would be redundant, and hides the caption entirely.

Changes:

• Modified MediaViewModel to use the full filename instead of DisplayName, track when details are loaded, and automatically hide captions that duplicate the title
• Updated LibraryService to respect already-loaded media details when applying cached properties
• Adjusted PlayerPage layout to vertically center the title area in MiniPlayer and Hidden states when the subtitle is hidden

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
Screenbox.Core/ViewModels/MediaViewModel.cs	Added DetailsLoaded property tracking, changed constructor to use full filename, added logic to hide redundant captions after loading details, and improved null safety by initializing string properties with string.Empty
Screenbox.Core/Services/LibraryService.cs	Added check to prevent overwriting media properties from cache if details were already loaded
Screenbox/Pages/PlayerPage.xaml	Added RowSpan and VerticalAlignment setters to vertically center the title area in MiniPlayer and Hidden states

[United600]

Don't like how it looks when it's by itself on the miniplayer. Should we consider increasing its size in that case? Also, why is the file extension missing?

[United600]

BTW, I'm working on a branch that adds a TitleBar control and I'm considering separating the PlayerPage titlebar from the miniplayer.

[huynhsontung]

Don't like how it looks when it's by itself on the miniplayer. Should we consider increasing its size in that case?

That would require some extra UI logic and not just a simple visual state change. I want to avoid that.

I'm working on a branch that adds a TitleBar control and I'm considering separating the PlayerPage titlebar from the miniplayer.

I think separating them is a good move. PlayerPage visual states are quite overloaded. The more reason we shouldn't invest too much into customizing the current mini title.