-
-
Notifications
You must be signed in to change notification settings - Fork 340
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(cpn): add support for surface radios inc Radiomaster MT12 #5381
Conversation
Add a very brief look, on the simu, the left stick can only be moved horizontally and does steering, good, but the right one that does throttle, is also only horizontal, which makes it really difficult for user to understand which one is which. Could the right one made to only move vertically ? |
@philmoz also raised this. Yes it can be re-orientated. |
Reading from radio and writing back to radio seems to work ok. No obvious issues with conversion. |
I think the LCD and buttons could be bigger. I had a look at the LCD code and it seems it could do scaling based on widget size (rather than just x2): Patch for this (have not tested if it affects other simulators yet): |
If we enlarge the lcd then buttons could also be enlarged and widened to take into account each OS font. |
Need to test layout for the new screen size you have been working on for a while so we have the full range covered. |
ed2443b
to
3c5f583
Compare
I'm also considering custom surface widgets for throttle/brake and steering to substitute for the air joysticks. Will need to determine how much effort and float mock up with surface users. |
If you are happy with this as it is, fine with me :) I'm not able to do much for the next few days on project so won't catch anything more than blindingly obvious issues. Just downloading the PR build now, will merge shortly if nothing immediate jumps out. |
Thanks for all the work on this! will test this weekend. |
Kills Global Functions/Special Functions switch data when copying model in companion MT12 |
Steps to reproduce please as this is too vague |
Fixes #4969
Summary of changes:
TODO