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

[BUG] Cabin temperature knobs not centered #1763

Closed
FlightSimmer7700 opened this issue Nov 4, 2020 · 6 comments · Fixed by #6921
Closed

[BUG] Cabin temperature knobs not centered #1763

FlightSimmer7700 opened this issue Nov 4, 2020 · 6 comments · Fixed by #6921
Labels
Bug Something isn't working

Comments

@FlightSimmer7700
Copy link
Contributor

Latest master

built: 2020-11-04T16:48:59+00:00
ref: refs/heads/master
sha: dbcd6ac
actor: Benjozork
event_name: push

Describe the bug
The cabin knobs are not fully centred and the "detents" are a little off to both the left and right

To Reproduce

1.Spawn in
2.look at overhead
3.turn the cabin temperature knobs using the mouse scroll wheel

Expected behaviour

It would look better if it was in the middle and not a little to either side

Actual behaviour

It stops at either side of the middle

References

Screenshot (163)
Screenshot (164)

Additional context

Was this working before/when did the issue start occurring?
I can't remember if it has always been like this or not just noticed it today

Is this a problem in the vanilla unmodded game?
Not sure but shouldn't matter

Discord username (if different from GitHub): FlightSimmer7700#3468

@FlightSimmer7700 FlightSimmer7700 added the Bug Something isn't working label Nov 4, 2020
@Pleasure0102 Pleasure0102 added the Texture A texture is wrong somewhere/a texture is missing (should be added) label Nov 4, 2020
@Pleasure0102 Pleasure0102 self-assigned this Nov 4, 2020
@lukecologne lukecologne removed the Texture A texture is wrong somewhere/a texture is missing (should be added) label Nov 4, 2020
@AdamNZ
Copy link

AdamNZ commented Nov 5, 2020

This cosmetic "bug" has been annoying me [slightly] ever since they got tweaked a few builds ago. Would be nice to have them looking all neat and tidy!

@lukecologne
Copy link
Member

So, the current template uses an increment of 4 with the mousewheel. Since the value goes from 0 to 100, the problem is obviously that you can't reach 50 within increments of 4. There are 2 ways of fixing this: either, increase the increment from 4 to 5, which would speed up the mousewheel scrolling, or, the second option, you can just use the mouse button to center it, since it has an increment of 1, it can reach the middle.

@Benjozork
Copy link
Member

@lukecologne, the number of steps needs to be uneven in order for a center position to exist. the only way is to make it either 99 or 101 states.

@lukecologne
Copy link
Member

lukecologne commented Nov 5, 2020

@Benjozork well, the number of states is uneven. You have to include the 0, which makes it a total of 101 states

@ImenesFBW
Copy link
Contributor

#3118 may inadvertently fix #1763 since the position is centered at the start, when the user moves it back and forth by the increment set they should be able to hit the centered position.

@rare-potato
Copy link
Contributor

I'm considering fixing this "bug" while working on issue #3128 to allow finer variability in initial air conditioner knob positioning.

The main downside to fixing this is going to be decreasing the speed at which a user can change the knob position with the mouse wheel, but it allows each location of the marked notches to be centered with the knob itself from what I've been able to observe.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants