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

Option to automatically adjust resolution when cursor size is changed #90

Closed
TiagoTiago opened this issue Dec 12, 2022 · 6 comments
Closed
Assignees
Labels
enhancement New feature or request

Comments

@TiagoTiago
Copy link

I understand that there may be situations where having a different resolution than the cursor size might be necessary; but it is inconvenient to have to keep setting things twice when you are ok with the resolutions matching; so it would be useful to have a checkbox to make both settings stay in sync.

@TiagoTiago TiagoTiago added the enhancement New feature or request label Dec 12, 2022
@zero01101
Copy link
Owner

i'm inclined to agree that it should be a toggleable option; i'll look into this :)

@zero01101
Copy link
Owner

so it's trivial enough to add a checkbox and logic to dump the matched reticle size value to the SD resolution setting, but the actual slider element is giving me grief lol :D

@seijihariki if there's a way to call a slider's setValue programmatically, i must be missing how to do so :/ any hints?

@seijihariki
Copy link
Collaborator

seijihariki commented Dec 12, 2022

so it's trivial enough to add a checkbox and logic to dump the matched reticle size value to the SD resolution setting, but the actual slider element is giving me grief lol :D

@seijihariki if there's a way to call a slider's setValue programmatically, i must be missing how to do so :/ any hints?

You should check the brush sliders because we set their value from the mouse wheel. This also triggers value callbacks, so changing the slider programmatically should also automatically set the new value to the stablediffusiondata object.

@seijihariki seijihariki removed their assignment Dec 12, 2022
@zero01101
Copy link
Owner

think i overcame my terminal stupidity and got this working! @TiagoTiago if you'd care to git checkout i90 and see if this is what you were thinking of :)

@TiagoTiago
Copy link
Author

TiagoTiago commented Dec 13, 2022

Yep, exactly like that, thanx :)


Erm, sorry I didn't reply sooner, I thought I had clicked the submit button earlier, got distracted...

@seijihariki
Copy link
Collaborator

This feature should be complete. Closing this issue for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants