diff --git a/briar-android/res/layout/fragment_contact_list.xml b/briar-android/res/layout/fragment_contact_list.xml index 5f1bb33709f91ac590c48bfdbc0a3f509675bd7a..787d4dff960f06c4f8d8b0c4d56fe73aa61baef0 100644 --- a/briar-android/res/layout/fragment_contact_list.xml +++ b/briar-android/res/layout/fragment_contact_list.xml @@ -1,27 +1,6 @@ <?xml version="1.0" encoding="utf-8"?> -<android.support.design.widget.CoordinatorLayout - xmlns:android="http://schemas.android.com/apk/res/android" - xmlns:app="http://schemas.android.com/apk/res-auto" - android:id="@+id/coordinatorLayout" - android:layout_width="match_parent" - android:layout_height="match_parent"> - - <org.briarproject.android.util.BriarRecyclerView +<org.briarproject.android.util.BriarRecyclerView + xmlns:android="http://schemas.android.com/apk/res/android" android:id="@+id/contactList" android:layout_width="match_parent" android:layout_height="match_parent"/> - - <android.support.design.widget.FloatingActionButton - android:id="@+id/addContactFAB" - android:layout_width="wrap_content" - android:layout_height="wrap_content" - android:layout_gravity="bottom|end" - android:layout_margin="@dimen/margin_activity_horizontal" - android:src="@drawable/ic_add_white" - app:fabSize="normal" - app:elevation="4dp" - app:layout_anchor="@id/contactList" - app:layout_anchorGravity="bottom|right|end" - app:layout_behavior="org.briarproject.android.util.HideFabOnScrollBehavior"/> - -</android.support.design.widget.CoordinatorLayout> diff --git a/briar-android/res/menu/contact_list_actions.xml b/briar-android/res/menu/contact_list_actions.xml new file mode 100644 index 0000000000000000000000000000000000000000..c5d0dc76ab40cd91b89391236e33f809d04f09a2 --- /dev/null +++ b/briar-android/res/menu/contact_list_actions.xml @@ -0,0 +1,12 @@ +<?xml version="1.0" encoding="utf-8"?> +<menu + xmlns:android="http://schemas.android.com/apk/res/android" + xmlns:app="http://schemas.android.com/apk/res-auto"> + + <item + android:id="@+id/action_add_contact" + android:icon="@drawable/ic_add_white" + android:title="@string/add_contact_title" + app:showAsAction="ifRoom"/> + +</menu> \ No newline at end of file diff --git a/briar-android/src/org/briarproject/android/contact/ContactListFragment.java b/briar-android/src/org/briarproject/android/contact/ContactListFragment.java index 117191f2205faa8437af3107b626428d7a9b4a11..08e2d2650e113ca409cddc3e4186f6a6b9a191b3 100644 --- a/briar-android/src/org/briarproject/android/contact/ContactListFragment.java +++ b/briar-android/src/org/briarproject/android/contact/ContactListFragment.java @@ -3,13 +3,15 @@ package org.briarproject.android.contact; import android.content.Intent; import android.os.Bundle; import android.support.annotation.Nullable; -import android.support.design.widget.FloatingActionButton; import android.support.v4.app.ActivityCompat; import android.support.v4.app.ActivityOptionsCompat; import android.support.v4.util.Pair; import android.support.v4.view.ViewCompat; import android.support.v7.widget.LinearLayoutManager; import android.view.LayoutInflater; +import android.view.Menu; +import android.view.MenuInflater; +import android.view.MenuItem; import android.view.View; import android.view.ViewGroup; @@ -108,6 +110,9 @@ public class ContactListFragment extends BaseFragment implements EventListener { @Override public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) { + + setHasOptionsMenu(true); + View contentView = inflater.inflate(R.layout.fragment_contact_list, container, false); @@ -148,21 +153,27 @@ public class ContactListFragment extends BaseFragment implements EventListener { list.setEmptyText(getString(R.string.no_contacts)); list.periodicallyUpdateContent(); - // Show a floating action button - FloatingActionButton fab = - (FloatingActionButton) contentView.findViewById( - R.id.addContactFAB); + return contentView; + } - // handle FAB click - fab.setOnClickListener(new View.OnClickListener() { - @Override - public void onClick(View v) { - startActivity(new Intent(getContext(), - KeyAgreementActivity.class)); - } - }); + @Override + public void onCreateOptionsMenu(Menu menu, MenuInflater inflater) { + inflater.inflate(R.menu.contact_list_actions, menu); + super.onCreateOptionsMenu(menu, inflater); + } - return contentView; + @Override + public boolean onOptionsItemSelected(final MenuItem item) { + // Handle presses on the action bar items + switch (item.getItemId()) { + case R.id.action_add_contact: + Intent intent = + new Intent(getContext(), KeyAgreementActivity.class); + startActivity(intent); + return true; + default: + return super.onOptionsItemSelected(item); + } } @Override diff --git a/briar-android/src/org/briarproject/android/util/HideFabOnScrollBehavior.java b/briar-android/src/org/briarproject/android/util/HideFabOnScrollBehavior.java deleted file mode 100644 index 9d847b9c2b06f0fbd160fd82383ed1bbbd78e4f0..0000000000000000000000000000000000000000 --- a/briar-android/src/org/briarproject/android/util/HideFabOnScrollBehavior.java +++ /dev/null @@ -1,42 +0,0 @@ -package org.briarproject.android.util; - -import android.content.Context; -import android.support.design.widget.CoordinatorLayout; -import android.support.design.widget.FloatingActionButton; -import android.support.v4.view.ViewCompat; -import android.util.AttributeSet; -import android.view.View; - -public class HideFabOnScrollBehavior extends FloatingActionButton.Behavior { - - public HideFabOnScrollBehavior(Context context, AttributeSet attrs) { - super(); - } - - @Override - public boolean onStartNestedScroll(CoordinatorLayout coordinatorLayout, - FloatingActionButton child, View directTargetChild, View target, - int nestedScrollAxes) { - return nestedScrollAxes == ViewCompat.SCROLL_AXIS_VERTICAL || - super.onStartNestedScroll(coordinatorLayout, child, - directTargetChild, target, - nestedScrollAxes); - } - - @Override - public void onNestedScroll(CoordinatorLayout coordinatorLayout, - FloatingActionButton child, - View target, int dxConsumed, int dyConsumed, int dxUnconsumed, - int dyUnconsumed) { - super.onNestedScroll(coordinatorLayout, child, target, dxConsumed, - dyConsumed, dxUnconsumed, - dyUnconsumed); - - if (dyConsumed > 0 && child.getVisibility() == View.VISIBLE) { - child.hide(); - } else if (dyConsumed < 0 && child.getVisibility() != View.VISIBLE) { - child.show(); - } - } - -} \ No newline at end of file