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

Navigation Bar #35

Merged
merged 19 commits into from
Oct 3, 2014
Merged

Navigation Bar #35

merged 19 commits into from
Oct 3, 2014

Conversation

ctsims
Copy link
Member

@ctsims ctsims commented Sep 30, 2014

Navigation bar implementation wired in. Interacts properly with the progress bar now so that it can be enabled/disabled and takes the place of the progress bar when enabled.

Conflicts:
	res/values/colors.xml
Fixed behavior for "Finish" button popping in too late on question lists with optional questions

Updated icons to proper flatness.

Introduced a depression state on the navigation buttons to signify "presses"
…ether it will exit the form if conditional repeats are followed
Conflicts:
	res/values/styles.xml
	src/org/odk/collect/android/activities/FormEntryActivity.java
	src/org/odk/collect/android/views/ODKView.java
android:gravity="center"
android:paddingBottom="1dp"
android:paddingTop="1dp"
android:text="DESHYDRATION" >
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this supposed to be here? Though I assume it gets overwritten elsewhere so it doesn't really matter.

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh, good point. Bad form regardless of overwrite. Fixing

@orangejenny
Copy link
Contributor

A couple of minor questions above, otherwise looks great to me.

@ctsims
Copy link
Member Author

ctsims commented Oct 3, 2014

Cool, I think I've addressed all of the comments but let me know if you
wanna keep chatting about some of the edge cases/judgement calls.

On Fri, Oct 3, 2014 at 11:05 AM, Jenny Schweers notifications@github.com
wrote:

A couple of minor questions above, otherwise looks great to me.


Reply to this email directly or view it on GitHub
#35 (comment)
.

orangejenny added a commit that referenced this pull request Oct 3, 2014
@orangejenny orangejenny merged commit 76c717a into master Oct 3, 2014
@orangejenny orangejenny deleted the nav_bar branch October 3, 2014 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants