-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
feat(efb): Set default efb auto brightness on #9632
base: master
Are you sure you want to change the base?
feat(efb): Set default efb auto brightness on #9632
Conversation
as part of the testing process, is there a need/steps to reset a user's settings? Or does a new build not read the existing settings? Curious if the testing is still valid if someone had already used a FBW aircraft and the old settings are already there. |
A new build does read existing settings as far as I’m aware. I tested this by loading the build and confirming the settings related to auto EFB brightness were applied on loading. I then set them to off, quit the flight, loaded back in and confirmed the settings were still set to off and user preference was therefore saved. |
I meant if I already used another build of the A380 and had the auto-brightness set to on. How would i know if the auto-brightness on is because of your code change, or because of my previous setup. Just bringing this up to consider adding more details/steps in the testing steps. If the EFB does read a users settings from another build, you might need to include testing steps to delete the previous saved config on their system before this test. |
Why does this touch the large files repo? For the setting, you can delete that specific key from the data store in the Coherent debugger |
Apologies I think I accidentally committed all from both my aircraft source repository and my large files repository. Should I make a new commit without the large files changes? |
a546da2
to
9799bd6
Compare
9799bd6
to
c33c46a
Compare
@tracernz I've tidied this PR up by removing the unnecessary commits including the accidental commit relating to the large files repo |
Fixes #9630
Summary of Changes
Updated EFB so auto brightness defaults to on
Screenshots (if necessary)
References
Additional context
Discord username (if different from GitHub):
MrJigs
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.