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 weapon steadiness bar not working #70060

Merged
merged 1 commit into from
Dec 8, 2023

Conversation

OldFriendEr
Copy link
Contributor

fix weapon steadiness bar not working

Summary

Bugfixes "fix weapon steadiness bar not working"

Purpose of change

Due to a bug the bar only shows up when steadiness is full. Fixes #69954.

Describe the solution

Problem was that the calculated steadiness to display was wrongly stored in an int while the calculated value was a double resulting in either empty (0) or full bar (1).

Describe alternatives you've considered

I actually thought that the problem would be way harder to solve

Testing

I created a default world spawned myself a gun and now steadiness fills up correctly

Additional context

# |<-- Using around 50, Maximum 72 Characters -->|

due to a bug the bar only shows up when steadiness is full, problem was
that the calculated steadiness was stored in an int while the value was
a double resulting in either empty or full bar.
# |<----      Limit Each Line to a Maximum Of 72 Characters       ---->|

# Provide links or keys to any relevant tickets, articles or other resources
# Example: fixes CleverRaven#1234, closes CleverRaven#2345, resolves CleverRaven#3456, references CleverRaven#4567
fixes CleverRaven#69954
@github-actions github-actions bot added [C++] Changes (can be) made in C++. Previously named `Code` <Bugfix> This is a fix for a bug (or closes open issue) json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Dec 7, 2023
@Maleclypse Maleclypse merged commit 85896ff into CleverRaven:master Dec 8, 2023
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions <Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Weapon steadiness bar doesn't display partial steadiness, only empty or full
2 participants