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

Common - Add settings for additional progress bar information #8428

Merged
merged 9 commits into from
Oct 10, 2021

Conversation

LinkIsGrim
Copy link
Contributor

When merged this pull request will:

  • Add settings to show time left and % done in progress bar.

@LinkIsGrim LinkIsGrim force-pushed the additional-progressbar-info branch from 383b657 to 7c8865c Compare September 5, 2021 22:28
@PabstMirror PabstMirror added this to the 3.14.0 milestone Oct 7, 2021
@PabstMirror PabstMirror added the kind/enhancement Release Notes: **IMPROVED:** label Oct 7, 2021
@PabstMirror
Copy link
Contributor

I know grad trenches did something funny with ace's progressBar, but as far as I can tell this should work fine with it.

LinkIsGrim and others added 4 commits October 7, 2021 02:28
Co-authored-by: PabstMirror <pabstmirror@gmail.com>
missing full stop
@LinkIsGrim
Copy link
Contributor Author

I know grad trenches did something funny with ace's progressBar, but as far as I can tell this should work fine with it.

Didn't use this with that yet, but worst that can happen is not showing the extra info. I think they use a modified version of the progressBar function.

Copy link
Member

@jonpas jonpas left a comment

Choose a reason for hiding this comment

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

Why prefix all strings with Setting? Seems redundant and we don't do it anywhere else.

@LinkIsGrim
Copy link
Contributor Author

Was for keeping things consistent with existing strings. Will adjust.

@jonpas jonpas merged commit 1b31843 into acemod:master Oct 10, 2021
AndreasBrostrom pushed a commit to AndreasBrostrom/ACE3 that referenced this pull request Dec 3, 2021
@LinkIsGrim LinkIsGrim deleted the additional-progressbar-info branch June 14, 2023 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement Release Notes: **IMPROVED:**
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants