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

Actions Buttons improvements: Tablet only flight #1099

Merged
merged 5 commits into from
Sep 29, 2014

Conversation

arthurbenemann
Copy link
Member

Improved the functionality of #1090

So the user can arm the copter via the tablet later, which is not possible if the copter is on LAND or RTL modes
@m4gr3d
Copy link
Member

m4gr3d commented Sep 29, 2014

@arthurbenemann I was planning to tackle this issue along with issue #1092.
Instead of having one specific disarm button, we should have a generic anti button.
So for example:

  • clicking on connect causes disconnect button and arm button to appear (connect replaced by arm)
  • clicking on arm causes disarm button and takeoff button to appear (disconnect replaced by disarm and arm replaced by takeoff)
  • clicking on takeoff causes the in flight action bar to appear. No anti button is needed here since you can land by clicking the land flight mode action button.

Since it's part of the guided flight mode
…Guided variable

Mainly the altitude should be set correctly.
@arthurbenemann
Copy link
Member Author

@ne0fhyk This achieves a similar result, without changing the structure that's in place now. I don't think there is a need for a disconnect button.

screenshot_2014-09-29-12-38-07

@m4gr3d
Copy link
Member

m4gr3d commented Sep 29, 2014

@arthurbenemann an user was complaining a few weeks back about not being able to locate the disconnect option and wanted us to implement a fake menu to always hint at it.
It's likely then this has occurred to several more users since the majority of the devices running DP are samsung. Adding a disconnect button would solve that issue nicely without adding any kludge.

@m4gr3d
Copy link
Member

m4gr3d commented Sep 29, 2014

@arthurbenemann also the structure (ui at least) of the flight action buttons will be updated to match with Google's updated design guide.

@arthurbenemann
Copy link
Member Author

@ne0fhyk Disconnect isn't all that useful. Please give me a good reason to click the disconnect button?
And as we move to tablet only control, disconnect is one of the last things you want to happen.

In any case I'm merging this PR, as it's better than what we have currently. At least that was my experience when test flying some minutes ago.

arthurbenemann added a commit that referenced this pull request Sep 29, 2014
Actions Buttons improvements: Tablet only flight
@arthurbenemann arthurbenemann merged commit 6b64566 into DroidPlanner:master Sep 29, 2014
@arthurbenemann arthurbenemann deleted the ActionButtons branch September 29, 2014 21:23
@squilter
Copy link
Member

squilter commented Oct 1, 2014

We now have 2 "connect" buttons. Would it make sense to remove the top one?

@m4gr3d
Copy link
Member

m4gr3d commented Oct 1, 2014

That makes sense on the flight screen, but the top one still needs to be available on the other screens.
Perhaps just set its showAsAction to never, this way it'll only show in the drop down menu.

@arthurbenemann
Copy link
Member Author

I'll do that.

@m4gr3d
Copy link
Member

m4gr3d commented Oct 1, 2014

On that note, is it crucial to have the action bar telemetry info visible in all the app screens?
Not having it in the Editor screen would free up valuable real estate in the action bar where the editor tools could be moved up.

@arthurbenemann
Copy link
Member Author

I'm making the fixes here: #1109

@ne0fhyk I like having the telemetry information being consistent thought out the app. But you make a great point about the real state for things like send and receive waypoints in the editor.

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.

3 participants