Skip to content
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

Loading rules of video vs gif is different between plugins #178

Closed
Markussss opened this issue Nov 16, 2016 · 5 comments
Closed

Loading rules of video vs gif is different between plugins #178

Markussss opened this issue Nov 16, 2016 · 5 comments

Comments

@Markussss
Copy link
Contributor

I just wanted to point out that the rules in plugins are not consistent. Especially between gfycat and imgur.

With «Zoom Videos» in options disabled.

  • When hovering over an imgur-link:
    • pointing to a gifv/mp4/webm, we get no video.
    • pointing to a gif, we get a gif
  • When hovering over a gfycat-link
    • pointing to a mp4/webm/no file extension, we get a gif
    • pinting to a gif, we get a gif

I believe one of these plugins should be updated with the "correct" thing to do, but I'm unsure what is correct as both of these can serve both videos and gifs of the same content, depending on what the user (or the addon) asks for.

@extesy extesy added the bug label Nov 16, 2016
@TheMikeyR
Copy link

TheMikeyR commented Oct 11, 2017

I have the same issue with imgur
example where hover doesn't work

@Zenivo
Copy link

Zenivo commented Apr 13, 2018

Still an issue with the latest version.
For example, this link doesn't work: http://i.imgur.com/uvfMyb3.gifv

@User198263321
Copy link

webm links are flat out not working for me

@extesy
Copy link
Owner

extesy commented Jul 14, 2020

webm links are flat out not working for me

Not helpful without any information on how to reproduce.

@LiliaDoe
Copy link
Contributor

LiliaDoe commented Nov 1, 2024

Both those links work fine for me on the latest version of HZ+(1.0.221) and Firefox (132.0)

@extesy extesy added can't reproduce and removed bug labels Nov 1, 2024
@extesy extesy closed this as not planned Won't fix, can't repro, duplicate, stale Nov 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants