search

Maurdekye / training-picker

Extension for stable-diffusion-webui that allows the user to mux through the keyframes of a video, and automatically pick and export training examples from individual keyframes.
109 stars 15 forks source link

16:9 videos on a 1080p monitor cause the page to scroll, making resizing the crop brush awkward and desyncing its position #1

Closed Maurdekye closed 2 years ago

Maurdekye commented 2 years ago

As the title says, if the image frame is too big to fit on a single screen and the page becomes scrollable, scrolling will cause the crop brush to desync from the mouse cursor.

Expected behavior:

The brush stays synced with the mouse cursor on scrolling / the image does not take up enough space to cause the page to become scrollable

Maurdekye commented 2 years ago

I am too lazy to fix this right now because it works fine on my 1440p monitor, but I may come back later to fix this if I can figure out how to manipulate the page layout properly (I am not an html/css expert 😛)

Maurdekye commented 2 years ago

Page now behaves better with regard to scrolling; the page will no longer scroll while the mouse is over the frame, and the crop brush stays properly synced if scrolling does occur.