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
Fix issue where 1080p means 720p #259
Conversation
It appears that the enumeration was not 100% the same in the settings causing 720p to be 640p, and 1080p to be 720p. This fixes xbmc#237
This is the relevant code that is affected by the settings choice. And why it is so important to keep both MAXRESOLUTION and MAXRESOLUTIONSECURE in sync. |
very cool, thx @dagwieers ! |
@peak3d Not to put any pressure on you, but is there a timeline when to expect this fix in the Kodi repositories? Looking at the feedback we received wrt. 720p not working, and the issues I found that could also be in part related to this issue (or at least troubled users in understanding what is going on), I think we shouldn't hold onto this too long. This impacts the Youtube and Netflix plugins as well, two popular addons. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Co-authored-by: Jaker MX <jakermx@users.noreply.github.com>
It appears that the enumeration was not 100% the same in the settings causing 720p to be 640p, and 1080p to be 720p.
This fixes #237 and add-ons/plugin.video.vrt.nu#195