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

Fix that negative integer values for float settings don't get a ".0" suffix #13779

Merged
merged 1 commit into from Sep 5, 2023

Conversation

grorp
Copy link
Member

@grorp grorp commented Sep 4, 2023

If you enter a positive integer value (or zero) for a float setting using the new settings GUI, it gets a .0 suffix. If you do the same with a negative integer value, it doesn't get a .0 suffix.

This PR fixes that. Both the feature and the bug were introduced with #13489.

To do

This PR is a Ready for Review.

How to test

Verify that float settings always get a .0 suffix if they have no decimal places

@grorp grorp added Trivial The change is a trivial bug fix, documentation or maintenance change, as per the Git Guidelines Bugfix 🐛 PRs that fix a bug @ Mainmenu labels Sep 4, 2023
Copy link
Member

@srifqi srifqi left a comment

Choose a reason for hiding this comment

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

This PR fixes the problem.

@grorp grorp merged commit 1a568cc into minetest:master Sep 5, 2023
2 checks passed
@grorp grorp deleted the fix-negative-whatever branch December 18, 2023 16:38
kawogi pushed a commit to kawogi/minetest that referenced this pull request Dec 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bugfix 🐛 PRs that fix a bug @ Mainmenu One approval ✅ ◻️ Trivial The change is a trivial bug fix, documentation or maintenance change, as per the Git Guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants