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

Adjust CMS and statistics page layout for HD OSD #7871

Merged
merged 3 commits into from
Mar 14, 2022

Conversation

geoffsim
Copy link
Contributor

@geoffsim geoffsim commented Mar 4, 2022

The CMS and statistics screens are cropped on the LHS when in HD 4:3 cut mode, making them unusable.
These changes adjust the columns of the elements to ensure the screens are fully visible when in HD mode.

(changes tested on both Analog and Digital HD systems)

Ensure the statistics screens are fully visible when in HD 4:3 cut.
@DzikuVx DzikuVx added this to the 5.0 milestone Mar 13, 2022
@DzikuVx
Copy link
Member

DzikuVx commented Mar 13, 2022

@geoffsim is this ready to merge?

@geoffsim
Copy link
Contributor Author

geoffsim commented Mar 13, 2022 via email

Copy link
Member

@DzikuVx DzikuVx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@geoffsim something to check/rethink/change

@@ -32,7 +32,7 @@
#include "platform.h"

FILE_COMPILE_FOR_SPEED

#define USE_OSD
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OSD should not define USE_OSD

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apologies for that - a temporary change that should not have been released.
I have removed it.

@DzikuVx DzikuVx merged commit 9499e78 into iNavFlight:master Mar 14, 2022
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