Requesting Additions to the Script.
|
|
Please add "body {overflow-x: hidden}" to end of CSS styles section.It removes the horizontal scrollbar when a overflow occurs on the body. The horizontal scrollbar is never required when using youtube so it can safely be removed. I requested this before however it seems to have gone unnoticed due to the topic it was in. Could you also please add an option to ignore aspect ratio in the videos. For example (From the resizeVideo function): case "max": // Fills the viewport dimensions completely, preserving the aspect ratio // Calculates the resize factor (factorR) as the smallest of the vertical and horizontal ratios // This is valid if the original video size is smaller than the viewport (at least in one dimension) var factorR = Math.min(vh / oPlayerData.height, vw / oPlayerData.width); // Multiplies the original dimensions by the factorR factor, rounded down (the rounding mustn't return a bigger number) // The aspect ratio is preserved newW = Math.floor(oPlayerData.width * factorR); newH = Math.floor(oPlayerData.height * factorR); break;would become
case "max":
if(preserveAspectRatio)
{
// Fills the viewport dimensions completely, preserving the aspect ratio
// Calculates the resize factor (factorR) as the smallest of the vertical and horizontal ratios
// This is valid if the original video size is smaller than the viewport (at least in one dimension)
var factorR = Math.min(vh / oPlayerData.height, vw / oPlayerData.width);
// Multiplies the original dimensions by the factorR factor, rounded down (the rounding mustn't return a bigger number)
// The aspect ratio is preserved
newW = Math.floor(oPlayerData.width * factorR);
newH = Math.floor(oPlayerData.height * factorR);
}
else
{ // aspect ratio is ignored
newW = vw;
newH = vh;
}
break;
Plus code for the aspect ratio option etcetera
Also the script update doesn't work properly, every time I click HIde (which should hide it for the session), it asks me again for the next video. This is due to the unnecessary else block in the script check.
else {
// If a new version was previously detected the notice will be shown to the user
// This is to prevent that the notice will only be shown once a day (when an update check is scheduled)
if (scriptLastRemoteVersion > scriptVersion) {
scriptShowUpdateMessage(true, scriptLastRemoteVersion);
}
}
Please remove the above code to fix it OR fix it some other way.
Also please add the code in my post here: http://userscripts.org/topics/22027#post-body-9... Much obliged & thank you, if these things are implemented then I can stop having to manually update the script every time there is an update. |
|
|
BUMP |
|
|
Please, don't bump threads, specially if it was posted only a few days before. It's rude and it won't get my attention faster. Besides I had already read your post, but after I few tests I discover a subtle bug in a support function of the script that I need to fix first. I'll answer you when I resolve the issue. Be patient! |
|
|
Ok, some answers:
|