Skip to content

Conversation

@Bnyro
Copy link
Contributor

@Bnyro Bnyro commented Feb 1, 2026

This is a follow-up to #2454 by @phisher98.

I noticed that it's problematic that currently, there are two text views displaying video info:

The second issue I noticed is that the video title is as much highlighted as the playback info, i.e. they use the same text size and color. This is problematic because the video title is much more important that such meta information and thus should be visually emphasized more than these statistics that most users probably don't need that often.

(For people that don't know what I'm talking about - it's about the "Vidoza [Deutsch] - Black Mirror - ..." text at the top in the screenshots)

Screenshot on my phone:
video-info-demo

TV layout:
Screenshot from 2026-02-01 12-47-12

Player/Trailer layout:
Screenshot from 2026-02-01 12-47-51

@phisher98
Copy link
Contributor

phisher98 commented Feb 1, 2026

Just a FYI I have a PR after the mentioned PR with Settings to hide and other changes to video display resolution #2464

@Bnyro
Copy link
Contributor Author

Bnyro commented Feb 1, 2026

I have a PR after the mentioned PR with Settings to hide and other changes to video display resolution #2464

  1. I don't see why we need a preference to selectively hide some of that information? Perhaps it makes sense to add a setting to either enable or disable all of these information items, but e.g. only showing the video codec but not the audio codec doesn't make much sense.
  2. Your PR doesn't include any visual changes as the ones I explained in the PR description.
  3. Showing the amount of channels used is nice addition though, very nice idea 👍

Anyways, it should be easy enough to rebase the other PR once one of our PRs is merged.

@phisher98
Copy link
Contributor

I have a PR after the mentioned PR with Settings to hide and other changes to video display resolution #2464

  1. I don't see why we need a preference to selectively hide some of that information? Perhaps it makes sense to add a setting to either enable or disable all of these information items, but e.g. only showing the video codec but not the audio codec doesn't make much sense.
  2. Your PR doesn't include any visual changes as the ones I explained in the PR description.
  3. Showing the amount of channels used is nice addition though, very nice idea 👍

Anyways, it should be easy enough to rebase the other PR once one of our PRs is merged.

Still learning, my brain is in beta. 😎

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