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.

Sign up for GitHub

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

Jump to bottom

Append Ares' SW.Shots usage to sw tooltip #1392

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Append Ares' SW.Shots usage to sw tooltip #1392

wants to merge 1 commit into from
+63 −24

Conversation

chaserli
Copy link
Contributor

@chaserli chaserli commented Sep 23, 2024 •
edited
Loading

uimd.ini[ToolTips]->SWShotsFormat csf entry default to %d/%d shots for now

@chaserli chaserli added the Minor Documentation is not required label Sep 23, 2024
@Phobos-developers Phobos-developers deleted a comment from github-actions bot Sep 23, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 23, 2024 •
edited
Loading

Nightly build for this pull request:

This comment is automatic and is meant to allow guests to get latest nightly builds for this pull request without registering. It is updated on every successful build.

@Metadorius Metadorius self-assigned this Sep 29, 2024
@Metadorius
Copy link
Member

Metadorius commented Sep 29, 2024 •
edited
Loading

Can you show a pic how it looks right now with default settings?

@chaserli
Copy link
Contributor Author

chaserli commented Oct 1, 2024

Can you show a pic how it looks right now with default settings?

image

@Metadorius
Copy link
Member

Hmm what if we wrote Name (7/10) instead? seems intuitive enough and not overloaded much to me

@chaserli
Copy link
Contributor Author

chaserli commented Oct 1, 2024

Hmm what if we wrote Name (7/10) instead? seems intuitive enough and not overloaded much to me

Then you can simply overwrite it in csf. You can use only 1 number if you want.
The c-style parsing as well as the default str was suggested by others.
I understand what you were thinking. You have a better word for it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Metadorius Metadorius

Labels
Minor Documentation is not required Tested
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@chaserli @Metadorius