Skip to content

feat(player): show original bitrate in quality controls#1534

Open
RyanTheTechMan wants to merge 2 commits into
edde746:mainfrom
RyanTheTechMan:feature/show-original-bitrate
Open

feat(player): show original bitrate in quality controls#1534
RyanTheTechMan wants to merge 2 commits into
edde746:mainfrom
RyanTheTechMan:feature/show-original-bitrate

Conversation

@RyanTheTechMan

Copy link
Copy Markdown
Contributor

Changes

  • Show the media's bitrate beside Original in the Quality Picker and Playback Settings menus.

Display the media's source bitrate beside Original in the quality picker and Playback Settings.
Copilot AI review requested due to automatic review settings July 11, 2026 05:02

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@edde746

edde746 commented Jul 11, 2026

Copy link
Copy Markdown
Owner

Could you include screenshots?

When multiple media versions are available, the version label already includes the source bitrate. There is no need to show it twice.
@RyanTheTechMan

Copy link
Copy Markdown
Contributor Author

Playing media with one version

CleanShot 2026-07-11 at 15 30 59 CleanShot 2026-07-11 at 15 31 02

Playing media with two versions

CleanShot 2026-07-11 at 15 30 39 CleanShot 2026-07-11 at 15 30 46

I also had the idea of adding parentheses around the bitrates like you have for converted versions.
That would apply for all 3 menus, Playback Settings, Quality picker, and Quality controls.
Example:
1080p (20 Mbps)
1080p (12 Mbps)
1080p (10 Mbps)

rather than:
Example:
1080p 20 Mbps
1080p 12 Mbps
1080p 10 Mbps

to bring consistency to the UI.
Let me know if I should make that part of this pull request.

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.

3 participants