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

Preview & Load not dismissed when Load is pressed #2989

Closed
philrz opened this issue Jan 25, 2024 · 1 comment · Fixed by #2991
Closed

Preview & Load not dismissed when Load is pressed #2989

philrz opened this issue Jan 25, 2024 · 1 comment · Fixed by #2991
Assignees
Labels
bug Something isn't working

Comments

@philrz
Copy link
Contributor

philrz commented Jan 25, 2024

tl;dr

A bug with the Preview & Load screen appeared at commit a5ee8af such that a load operation is initiated when the Load button is pressed but the screen does not close until the load completes.

Details

I confirmed via binary search that this bug first appeared at commit a5ee8af (#2955).

As shown in the attached video, when I tee up a large set of files in Preview & Load and press the Load button, the load does begin (you can see the tab appear in the background). But since I'm accustomed to the screen being dismissed and watching the filling load thermometer, I ended up hitting the button again multiple times thinking I maybe mis-clicked and it ends up initiating repeated additional loads of the same data to even more pools.

Repro.mp4
@philrz philrz added the bug Something isn't working label Jan 25, 2024
@philrz
Copy link
Contributor Author

philrz commented Jan 26, 2024

Verified in Zui commit 7dd553b.

As shown in the attached video, now when I click the Load button the Preview & Load screen is immediately dismissed and I can watch the rest of the load proceed by watching the thermometer fill up on the pool summary page.

Verify.mp4

Thanks @jameskerr!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants