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

[Bug] Fix tab indexes are reseted after restart UI #14637

Conversation

light-and-ray
Copy link
Contributor

@light-and-ray light-and-ray commented Jan 13, 2024

Description

When you selected in web browser tab img2img->Resize by, or Extras->Batch Process, then you restart webui, the internal state of tabs' indexes are reseted. It also happened in one my extension, where I used the same pattern of tab_index, it is really annoying

I've found it happens because of gr.State component. I've changed it on invisible gr.Number, and the bug is gone. So I decided to make a PR

Checklist:

.gitignore Outdated
@@ -38,3 +38,4 @@ notification.mp3
/package-lock.json
/.coverage*
/test/test_outputs
.directory
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A file from kde's Dolphin file manager, contains directory's metadata

@light-and-ray light-and-ray changed the title [Bug] Fix tab indexes reset after restart UI [Bug] Fix tab indexes are reseted after restart UI Jan 13, 2024
@AUTOMATIC1111
Copy link
Owner

.directory should go into your own .git/info/exclude. Remove that and I'll merge the PR.

@light-and-ray light-and-ray force-pushed the fix_tab_indexes_resets_after_restart_ui branch from 46fea31 to 56676ff Compare January 20, 2024 07:49
@light-and-ray
Copy link
Contributor Author

@AUTOMATIC1111 okay, thanks. Fixed

@AUTOMATIC1111 AUTOMATIC1111 merged commit 1dbee39 into AUTOMATIC1111:dev Jan 20, 2024
3 checks passed
@light-and-ray light-and-ray deleted the fix_tab_indexes_resets_after_restart_ui branch January 20, 2024 10:38
@w-e-w w-e-w mentioned this pull request Feb 17, 2024
@pawel665j pawel665j mentioned this pull request Apr 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants