-
Notifications
You must be signed in to change notification settings - Fork 58
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
[Suggestion] - About the problem with redgifs #586
Comments
What problem? |
can you give an example link? |
all links in redgif. any. |
I open redgifs - click any video and it works fine for me |
try disabling other scripts or extensions which may be breaking this, |
I notice two problems:
I receive the error
|
none of this If you want try a CLEAN Firefox portable, which I'm using for testing, you can download it here, I simply installed the script and added the two in the bookmark bar |
the video url is definitely wrong there, try the new script version - it now waits for the video to load before grabbing it |
nothing changed, |
It's not the firefox which is different but the website's content try clicking "Inspect" on the video element when the script is not running (disable it or press the new script version should solve this by avoiding such video urls to grab |
it doesn't activate since there is no proper video to grab and use (else there'd be that error you've seen), only a |
see this, I don't know, work better the other script, with that I see how it should be. |
if you mean the style script above - yea, there's no reason it wouldn't work - it removes everything around the video without changing anything unlike here and this script creates a whole new page (to avoid nasty ads and pop-ups), I can't think of a way to fix this right now |
should work again now, |
about the problems with redgif, mime error etc etc
On reddit there is a script that work very well, maybe can be implemented on handyimage
// ==UserScript==
// @name Redgifs full height video
// @match https://www.redgifs.com/watch/*
// @grant none
// @Version 1.0
// @run-at document-start
// ==/UserScript==
'use strict';
function addCSS(text) {
const el = document.createElement('style');
el.innerHTML = text;
}
addCSS(
.previewFeed:has(.Player.Player_isActive){position: fixed;margin: 0;inset: 0;max-width: unset;overflow: hidden;> :not(.Player){display: none}> .Player{height: 100vh;max-height: unset;width: 100%;min-width: 100%;margin: 0;position: relative;.Video-ProgressBar{bottom: 0;transform: translateY(-18px)}video{z-index: 9}}}.App:has(.Player.Player_isActive) :is(.topNav, .ApplicationFooter, .Player-MetaInfo, .Player:not(.Player_isActive)){display: none}
);The text was updated successfully, but these errors were encountered: