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
4.93k stars 1.01k forks source link

fix(nd): fix active leg navaid color #8696

Closed BlueberryKing closed 5 days ago

BlueberryKing commented 6 days ago

Fixes #8691

Summary of Changes

Previously, when the active leg termination was a tuned navaid, the symbol appeared in white. This is incorrect, tuned navaids should appear in cyan. This PR fixes it.

Screenshots (if necessary)

image

References

see issue #8691

Additional context

There are more errors in the ND symbology. For example, symbols from the EFIS overlay should be behind flightplan symbols. I don't have the capacity to fix these issues at the moment, so I will restrict the PR to this small fix.

Discord username (if different from GitHub):

Testing instructions

How to download the PR for QA

Every new commit to this PR will cause new A32NX and A380X artifacts to be created, built, and uploaded.

  1. Make sure you are signed in to GitHub
  2. Click on the Checks tab on the PR
  3. On the left side, find and click on the PR Build tab
  4. Click on either flybywire-aircraft-a320-neo or flybywire-aircraft-a380-842 download link at the bottom of the page
LunakisDev commented 5 days ago

Quality Assurance Trainee Report

Discord: LunakisLeaks Object of testing: #8696 Tier of Testing: 1 Date: 26/6/2024

Testing Process:

  1. Spawned at LGAV RNW.
  2. Tuned navaids on the RAD NAV page + I make sure they appear with a cyan color.
  3. DIR TO Navaid and I ensure it still appears with a cyan color.
  4. Tested different combinations of EFIS filters.

Negatives: N/A

Testing Results: Passed

Conclusions: LGTM