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

Horizont dev #285

Merged
merged 3 commits into from
Jan 1, 2024
Merged

Horizont dev #285

merged 3 commits into from
Jan 1, 2024

Conversation

iltis42
Copy link
Owner

@iltis42 iltis42 commented Jan 1, 2024

New Horizon style to be seen here:
https://www.youtube.com/watch?v=xa4NbsjAgKo

moshe-braner and others added 3 commits December 31, 2023 17:06
- bigger, brighter, includes "airplane" icon, much lighter on the CPU.  Also changed some color names for consistency.
@iltis42 iltis42 merged commit ea39a38 into master Jan 1, 2024
2 checks passed
@moshe-braner
Copy link
Contributor

I think that the small airplane icon from the FLARM warning screen is (1) too small, and (2) confusing because it looks almost the same as the "from above" view (the other airplane icon in the FLARM warning). I have a one-icon FLARM warning screen PR ready to submit, but waiting for the RICO audio changes to be finalized, since the setup menu files are affected by both. There I have a larger airplane icon for the single-icon version - but it is "from above". So I still think that a "view from behind" airplane icon for the horizon screen needs to be implemented, that is both bigger and less confusing. Perhaps something like my two simple lines (wings and tail) but adding a round fuselage? Skipping the horizontal T-tail though, that is the confusing part.

@moshe-braner
Copy link
Contributor

Also, not clear if you want the "static demo" horizon display. The idea is so people will see that and think "yes, I like that, I should pay the extra for the AHRS license to make this come alive". To make it appear, need to change the code in SetupMenu system_menu_create_hardware_rotary_screens() to remove the if(license) conditional.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants