Remove autoarPreventConditions — they're redundant with new way of calculating AR and, as of now, interfere with netflix' ultrawide workarounds

This commit is contained in:
Tamius Han 2020-12-15 20:13:23 +01:00
parent 7881ae5290
commit 3d64a08b08
2 changed files with 12 additions and 26 deletions

View File

@ -406,6 +406,18 @@ const ExtensionConfPatch = [
}
}
}
}, {
forVersion: '4.5.0.1',
updateFn: (userOptions, defaultOptions) => {
for (const site in userOptions.sites) {
try {
delete userOptions[sites].autoarPreventConditions
} catch (e) {
// doesn't matter if site doesn't have that option,
// everything is still fine
}
}
}
}
];

View File

@ -1039,19 +1039,6 @@ var ExtensionConf = {
videoAlignment: VideoAlignment.Default,
keyboardShortcutsEnabled: ExtensionMode.Default,
arPersistence: true, // persist aspect ratio between different videos
autoarPreventConditions: { // prevents autoar on following conditions
videoStyleString: { // if video style string thing does anything of what follows
containsProperty: { // if video style string has any of these properties (listed as keys)
'height': { // if 'height' property is present in style attribute, we prevent autoar from running
allowedValues: [ // unless attribute is equal to anything in here. Optional.
'100%'
]
}
// 'width': true // this would prevent aard from running if <video> had a 'width' property in style, regardless of value
// could also be an empty object, in theory.
}
}
},
"DOM": {
"player": {
"manual": true,
@ -1071,19 +1058,6 @@ var ExtensionConf = {
videoAlignment: VideoAlignment.Default,
keyboardShortcutsEnabled: ExtensionMode.Default,
arPersistence: true, // persist aspect ratio between different videos
autoarPreventConditions: { // prevents autoar on following conditions
videoStyleString: { // if video style string thing does anything of what follows
containsProperty: { // if video style string has any of these properties (listed as keys)
'height': { // if 'height' property is present in style attribute, we prevent autoar from running
allowedValues: [ // unless attribute is equal to anything in here. Optional.
'100%'
]
}
// 'width': true // this would prevent aard from running if <video> had a 'width' property in style, regardless of value
// could also be an empty object, in theory.
}
}
},
DOM: {
"player": {
"manual": true,