New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add resolution text for 384 sized video #8212
Add resolution text for 384 sized video #8212
Conversation
I don't mind mereging this but I'd really be interested in an official source for this resolution. |
The resolution is mentioned in this Wikipedia article, so I guess it does exist in some form. https://en.wikipedia.org/wiki/Computer_display_standard |
After reviewing more videos, I'd like to also add 400p as a resolution text. Although not a broadcast or home media standard resolution, Scene standard definition releases have a maximum width of 720 and 16:9 aspect ratio results in the following resolutions:
Even though technically 404p would be a more accurate description, 400p is a nice round number and makes me feel warm and fuzzy. But I'd be ok either way. Let me know if it would be fine to update this PR, or if a separate PR would be preferred. |
Add resolution text for 384 sized video Original-merge: 7ffdde9 Merged-by: Cody Robibero <cody@robibe.ro> Backported-by: Joshua Boniface <joshua@boniface.me>
Add resolution text for 384 sized video Original-merge: 7ffdde9 Merged-by: Cody Robibero <cody@robibe.ro> Backported-by: Joshua Boniface <joshua@boniface.me>
Changes
Add 384p as a resolution text output option. This handles the somewhat common low resolution of 512x384. Meant for 10.8 branch.
Updates test.
Issues
Fixes #8099 - handles more another case anyway.
Builds on the work of #8104