flybywiresim / aircraft

The A32NX & A380X Project are community driven open source projects to create free Airbus aircraft in Microsoft Flight Simulator that are as close to reality as possible.
https://flybywiresim.com
GNU General Public License v3.0
5.02k stars 1.05k forks source link

[BUG] Cabin temperature knobs not centered #1763

Closed FlightSimmer7700 closed 2 years ago

FlightSimmer7700 commented 3 years ago

Latest master

built: 2020-11-04T16:48:59+00:00 ref: refs/heads/master sha: dbcd6ac1eee5a6ba875257ee88bede129ce6f91c 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

AdamNZ commented 3 years ago

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 commented 3 years ago

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 commented 3 years ago

@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 commented 3 years ago

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

ImenesFBW commented 3 years ago

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 commented 2 years ago

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.