a1111 webui's slider 8 incremental step wont work if [JavaScript aspect ratio controls] is activated

[bryanray]

commit: [a9fed7c3]

a1111 webui's slider 8 incremental step wont work if [JavaScript aspect ratio controls] is activated,

this perhaps is always the case :)

[bryanray]

javascript icons/ lengends are not centered as it should be usually, instead they are left aligned.

this particular issue appear in latest update :))

[thomasasfk]

Hey! The step was deliberately set to 1 so the aspect ratio could be as precise as possible.

It's very easy to make configurable though, so I'll add an option for it when I have time.

https://github.com/thomasasfk/sd-webui-aspect-ratio-helper/blob/50cf127c44fc100e3e78b84f288cb07f6c7dc67e/javascript/aspectRatioController.js#L79

In terms of the styling, it should be easy enough to center the text, I'll do that at the same time as the other change!

Thanks for raising.

[bryanray]

i wish i know how to code, then i can contribute to this plugin :) LOL

[bryanray]

Hey! The step was deliberately set to 1 so the aspect ratio could be as precise as possible.

It's very easy to make configurable though, so I'll add an option for it when I have time.

https://github.com/thomasasfk/sd-webui-aspect-ratio-helper/blob/50cf127c44fc100e3e78b84f288cb07f6c7dc67e/javascript/aspectRatioController.js#L79

In terms of the styling, it should be easy enough to center the text, I'll do that at the same time as the other change!

Thanks for raising.

another really important thing is, the "off" options should have been off to make 8 incremental stepping on again, that would be great!

[thomasasfk]

https://github.com/thomasasfk/sd-webui-aspect-ratio-helper/issues/36#issuecomment-1492479473

[bryanray]

thanks dude :)

amazing app you've done here