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

Progress bar or blocker when downloading mod list. #10904

Closed
2 tasks done
Caballero-Arepa opened this issue Jan 9, 2024 · 4 comments · Fixed by #10909
Closed
2 tasks done

Progress bar or blocker when downloading mod list. #10904

Caballero-Arepa opened this issue Jan 9, 2024 · 4 comments · Fixed by #10909
Labels

Comments

@Caballero-Arepa
Copy link
Contributor

Before creating

  • This is NOT a gameplay feature from Civ VI, BNW, or outside - see Roadmap
  • This is NOT a gameplay feature from Vanilla Civ V or from G&K - If so, it should be a comment in Missing features from Civ V - G&K #4697

Problem Description

I'm always annoyed when I want to go and search a mod with the search bar, and when I click enter, apparently the mod list hasn't finished downloading.
image

Related Issue Links

No response

Desired Solution

Display a loading bar, or perhaps more simple, a pop-up that tells you that the mod list is loading and to sit down and be quiet, but immediately upon opening the Mod Manager, not upon doing an action.

Alternative Approaches

Or don't display the mod list that has been downloaded. Only show the mod list when it has finished.

Additional Context

It's very late at night, I'm so uncivilized rn

@SomeTroglodyte
Copy link
Collaborator

I know. That message was a kludge to address an immediate problem quickly and was never meant to stay.
The LoadingImage widget from #10262 was to go in here too...

@SomeTroglodyte
Copy link
Collaborator

Fitting the indicator inside the header row it could look like this:

image

... or should I make it a giant one centered over the horizontal separator and almost reaching the top?

image
(oops my math is off target)

@Caballero-Arepa
Copy link
Contributor Author

Nah I think the small one is good

@Caballero-Arepa
Copy link
Contributor Author

Thank you very much

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

Successfully merging a pull request may close this issue.

2 participants