Nice Feature Idea - Scrub GIF about nsfwjs HOT 8 CLOSED

@GantMan don't be sorry :)
I can handle it, but it will take a while for me.

from nsfwjs.

Awesome! Off the top of my head here's a simple implementation:

• classifyGif takes options. One option is an object to store the SuperGif
• When you use this option, SuperGif is not set to null in classifyGif
• The web page now has a reference tot he SuperGif
• Victory Bar Charts can have an onHover event described in their docs. Add that to the existing GifBar
• The onHover calls the SuperGif move_to(i) - I think plus 1 - check index off by one possibility here

TADAAAAAAA 🎉

from nsfwjs.

Looks great, I updated your contributor status in the readme: 5413bd0

from nsfwjs.

@xilaraux - sorry I took your last ticket, would you want this one?

from nsfwjs.

You almost did it.
I will implement it. :)
I don't understand one thing. Should the frame be displayed next to this text or somewhat separately?

from nsfwjs.

Hahahaha.

So I think if you call GifReference.move_to(i) the GIF they dropped will lock to that frame. No need to show it anywhere. Basically it will stop animating and move to that frame. I think.... :) Once you start experimenting it should be clear. Then when you stop mousing over the graph, it would be cool if it went back to animating.

from nsfwjs.

@xilaraux - how goes it?

from nsfwjs.

@GantMan
You can see the progress here - #83
I want to propose to name SuperGif object as gifControl and setGitControl and prop in config. What do you think?
Also, I have doubts about the way I am setting this object and saving if to the React state, but spoiling the window is not an option neither.
Looking forward to your review :)
Sorry for the delay.

from nsfwjs.