c172p-team / c172p

A high detailed version of the Cessna 172P aircraft for FlightGear
GNU General Public License v2.0
82 stars 43 forks source link

Bugs in view logic and fuel priority #1377

Closed wlbragg closed 3 years ago

wlbragg commented 3 years ago

Fixes #1374 Fixes #1373

@legoboyvdlp Once merged these also need cherry picked into release 2020.3. I verified your first cherry pick it looks fine.

I'm tired of making issues for a simple bug fix so I am adding the fuel bug to this, this I changed the name. See the discussion at https://forum.flightgear.org/viewtopic.php?f=66&t=39392 if you have any questions as to why I changed it again. The PR you merged the other day technically fixe it but it didn't need all the superfluous code.

This is ready to go.

legoboyvdlp commented 3 years ago

Interesting. I relocated from SVVP to EGAA, when I did that, the gear reactions seemed to break, because at that point the aircraft simply refused to move.

Regardless; these two bugs are fixed - the aircraft engine will die when you crash and the view bugs are fixed.

wlbragg commented 3 years ago

Thanks, did you wait long enough after relocating? This is important to know if that bug is still there. It takes a few seconds after its back up after a relocate for the gear to release and the aircraft to move. If it did lock up then I know what the bug is and I will figure out how to fix it.

Megaf commented 3 years ago

Doesn't fix #1373

legoboyvdlp commented 3 years ago

I checked it specifically, and when I scrolled the zoom changed. So not sure why not.