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

Add user actions to Mission Control #1697

Merged
merged 1 commit into from
Jan 18, 2023

Conversation

MrD-RC
Copy link
Collaborator

@MrD-RC MrD-RC commented Jan 17, 2023

Mission Control updates

  • Added user actions
  • Added hover over text for P1 and P2 in actions

Demo

Examples of hover text
image

image

Mission Control updates
- Added user actions
- Added hover over text for P1 and P2 in actions
@MrD-RC MrD-RC added this to the 6.0 milestone Jan 17, 2023
@MrD-RC MrD-RC requested a review from stronnag January 17, 2023 21:13
@stronnag
Copy link
Collaborator

I tested User Actions will a prior example with WP#11 and WP#14 having user actions, one with relative and one with AMSL altitudes.

Screenshot from 2023-01-17 21-44-46
and
Screenshot from 2023-01-17 21-45-34

Which in the configurator, look like:
Screenshot from 2023-01-17 21-41-57
and
Screenshot from 2023-01-17 21-41-44

Ignoring implementation details like numbering philosophy and different altitude sources (try a 120 WP abs alt mission one day ....); they are functionally identical and generate equivalent mission files.

Setting and clearing bits as expected.

Copy link
Collaborator

@stronnag stronnag left a comment

Choose a reason for hiding this comment

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

Tested and functioning as expected.

@MrD-RC
Copy link
Collaborator Author

MrD-RC commented Jan 17, 2023

Thanks @stronnag

@MrD-RC MrD-RC merged commit 1eb76e0 into master Jan 18, 2023
@MrD-RC MrD-RC deleted the MrD_Add-user-actions-to-Mission-Control branch January 18, 2023 09:30
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