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

Make recording status more obvious #5486

Merged
merged 2 commits into from
Jul 31, 2017

Conversation

jaxxzer
Copy link
Collaborator

@jaxxzer jaxxzer commented Jul 21, 2017

We had a disappointing report of someone loosing a lot of footage they thought they were recording because they had accidentally set their hand on the touchpad or something and didn't notice they turned it off. In an effort to prevent these types of happenings, we are trying to make it more obvious when recording is active/inactive. This makes the recording button visibility toggle at 1Hz. I thought it might be distracting to do it this way, but I don't think it is after seeing it.

@dogmaphobic
Copy link
Contributor

There could be an option to start recording when armed (and stop when disarmed). That way you can't forget. I've done this so many times it's not even funny. Even on DJI's.

@DonLakeFlyer
Copy link
Contributor

@jaxxzer Can you rebase and I'll merge

@dheideman dheideman force-pushed the pr-blink-record-btn branch from 0573a34 to d6e31b0 Compare July 24, 2017 16:57
@jaxxzer
Copy link
Collaborator Author

jaxxzer commented Jul 24, 2017

Hi @DonLakeFlyer, I'm working with @dheideman on this and will let you know when we are finished with it.

@jaxxzer jaxxzer force-pushed the pr-blink-record-btn branch from d6e31b0 to c8e4878 Compare July 24, 2017 17:36
@jaxxzer
Copy link
Collaborator Author

jaxxzer commented Jul 24, 2017

This is good to go on our end, rebase complete.

@jaxxzer jaxxzer closed this Jul 26, 2017
@jaxxzer jaxxzer reopened this Jul 26, 2017
@DonLakeFlyer DonLakeFlyer merged commit 7ae2f7a into mavlink:master Jul 31, 2017
@dheideman dheideman deleted the pr-blink-record-btn branch July 31, 2017 15:05
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