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

Add a loading popup when opening the import/export/delete menus until the data sizes are calculated #372

Closed
gizellos opened this issue Jan 9, 2018 · 8 comments

Comments

@gizellos
Copy link

gizellos commented Jan 9, 2018

As the title says - opening Delete/Import/Export data pop out is too slow. I have quite fast computer and when I open it (for the first time), it takes a few seconds to open it.

My guess is that the pop out waits for all the sizes before it opens. Is it true? If it is, then I suggest first open the pop out and after it to get the sizes. Don't wait for the sizes. You can temporary show loading icons instead of the sizes.

It was not so slow earlier (I am not sure which version caused it).

@rafaelgomesxyz
Copy link
Owner

You would still need to wait to do something because the popup would freeze until all sizes are calculated. Is it faster if you open https://www.steamgifts.com/esgst/import, https://www.steamgifts.com/esgst/export and https://www.steamgifts.com/esgst/delete?

@rafaelgomesxyz
Copy link
Owner

rafaelgomesxyz commented Jan 10, 2018

I remember the popup always taking a few seconds for me (2-3). Maybe it was your data size got bigger?

@gizellos
Copy link
Author

Yes for me 2-3 seconds too and it is too much according to the fact that all other except the export/import open instantly. You could delay the calculating. Just don't execute the calculating until the pop out is fully loaded. Or calculate it on demand (a button called Calculate sizes)

Is it faster if you open https://www.steamgifts.com/esgst/import, https://www.steamgifts.com/esgst/export and https://www.steamgifts.com/esgst/delete?

It is the same.

@rafaelgomesxyz
Copy link
Owner

rafaelgomesxyz commented Jan 10, 2018

Like I said, it doesn't matter, you wouldn't be able to do anything because the popup would be frozen until the sizes are calculated. Well, that fact is irrelevant because the others don't need to calculate data sizes. And I don't think 2-3 seconds is too much at all, especially since you wouldn't use this that frequently for it to bother you that much, unless you import/export every day? But even then, waiting 2-3 seconds once per day doesn't sound like a big deal to me.

I will see if I can do something, but I already tried one thing and it remained the same, so I don't think there is anything to do. Are you sure it wasn't 2-3 seconds before? It has always been like that for me.

@gizellos
Copy link
Author

The time itself is not such a problem. It just a bit confusing and"stressful" when you click on something and nothing happens immediately.

I am not so sure that the time was better earlier, I just believe it because I was never thinking about it before. It can be because I have more data saved as you told me.

A loading icon could help it, though.

@rafaelgomesxyz rafaelgomesxyz changed the title Opening Delete/Import/Export data pop out is too slow Add a loading popup when opening the import/export/delete menus until the data sizes are calculated Jan 12, 2018
@rafaelgomesxyz rafaelgomesxyz added this to the 7.13.0 milestone Jan 12, 2018
@rafaelgomesxyz
Copy link
Owner

I just installed v7.11.2, which is the one before the version that caused slowdown for some things, and it takes the same amount of time for me.

But I added a loading popup to the latest dev version that gives the illusion that something is happening immediately. Let me know what you think.

@rafaelgomesxyz
Copy link
Owner

This also game me the idea for #385.

@gizellos
Copy link
Author

gizellos commented Jan 12, 2018

It's possible that it was faster only before you added the sizes for all sub options. So there is not a version which slowed it down. It is slower just because of the more info the script has to find out. So the issue is invalid, sorry.

But the loader you made is very good and it 100% helped it. The illusion works and I don't have the problem at all anymore. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants