Skip to content

Add menu to the betaflight script when used on OpenTx 2.3 tools screen #254

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

Merged
merged 7 commits into from
Oct 13, 2019

Conversation

frozenskys
Copy link
Contributor

@frozenskys frozenskys commented Oct 9, 2019

Adds a menu to the betaflight script when run from the tools screen, but not when run as a telemetry script. Needs #253 merged first.

menu_screen

@mikeller
Copy link
Member

mikeller commented Oct 9, 2019

Oooh, flash! 💯

mikeller
mikeller previously approved these changes Oct 10, 2019
@raphaelcoeffic
Copy link
Member

@frozenskys rebased as well. Can you please check I did not break anything? Please note you will have to "force pull", or "reset hard" on the remote ref to have the latest version and not effect a merge.

@raphaelcoeffic raphaelcoeffic changed the base branch from master to virtual-events October 10, 2019 10:34
@McGiverGim
Copy link
Member

Tested and it seems to work. To me is a little unintuitive to press exit to go the menu, I supposed this will be done with the menu key, but this key cycles the betaflight pages, like in the telemetry script. So it seems the menu is an intro page and the exis is a backward key. Is ok, only I need to use it more :)

@mikeller mikeller added this to the 1.5 milestone Oct 12, 2019
@mikeller mikeller merged commit 138e4e2 into betaflight:virtual-events Oct 13, 2019
@mikeller mikeller mentioned this pull request Oct 13, 2019
@frozenskys frozenskys deleted the tools-screen-menu branch October 13, 2019 11:32
mikeller added a commit that referenced this pull request Oct 15, 2019
Telemetry script fixes after #254
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.

4 participants