Skip to content

Commit

Permalink
Merge pull request owncloud#1141 from owncloud/559_longpress_filename
Browse files Browse the repository at this point in the history
Show file name in the contextual menu, see owncloud#559
  • Loading branch information
davivel committed Sep 11, 2015
2 parents 7ae7cef + e9f7937 commit 7182b1d
Show file tree
Hide file tree
Showing 3 changed files with 38 additions and 7 deletions.
27 changes: 23 additions & 4 deletions res/layout/file_actions.xml
Original file line number Diff line number Diff line change
Expand Up @@ -3,12 +3,31 @@
xmlns:android="http://schemas.android.com/apk/res/android"
android:layout_width="fill_parent"
android:layout_height="fill_parent"
android:layout_margin="@dimen/standard_margin">
android:layout_margin="@dimen/standard_margin"
android:orientation="vertical">

<TextView
android:id="@+id/file_actions_header"
android:layout_width="match_parent"
android:layout_height="wrap_content"
android:padding="@dimen/standard_padding"
android:textColor="@color/black"
android:visibility="gone"
/>

<View
android:id="@+id/file_actions_header_divider"
android:layout_width="match_parent"
android:layout_height="2dp"
android:background="@color/owncloud_blue"
android:visibility="gone"
/>

<ListView
android:id="@+id/file_actions_list"
android:layout_width="match_parent"
android:layout_height="match_parent" >
</ListView>
android:layout_height="0dp"
android:layout_weight="1"
/>

</LinearLayout>
</LinearLayout >
16 changes: 14 additions & 2 deletions src/com/owncloud/android/ui/dialog/FileActionsDialogFragment.java
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@
import android.widget.AdapterView.OnItemClickListener;
import android.widget.ArrayAdapter;
import android.widget.ListView;
import android.widget.TextView;

import com.owncloud.android.R;
import com.owncloud.android.ui.dialog.parcel.MenuItemParcelable;
Expand All @@ -27,6 +28,7 @@ public class FileActionsDialogFragment extends DialogFragment implements
OnItemClickListener {
private static final String ARG_ITEM_LIST = "ITEM_LIST";
private static final String ARG_FILE_POSITION = "FILE_POSITION";
private static final String ARG_FILE_NAME = "FILE_NAME";
public static final String FTAG_FILE_ACTIONS = "FILE_ACTIONS_FRAGMENT";

private List<MenuItemParcelable> mMenuItems;
Expand All @@ -48,7 +50,7 @@ public interface FileActionsDialogFragmentListener {
* @param menu menu to be display.
* @return Dialog ready to show.
*/
public static FileActionsDialogFragment newInstance(Menu menu, int filePosition) {
public static FileActionsDialogFragment newInstance(Menu menu, int filePosition, String fileName) {
FileActionsDialogFragment fragment = new FileActionsDialogFragment();
Bundle args = new Bundle();

Expand All @@ -57,6 +59,7 @@ public static FileActionsDialogFragment newInstance(Menu menu, int filePosition)

args.putParcelable(ARG_ITEM_LIST, menuParcelable);
args.putInt(ARG_FILE_POSITION, filePosition);
args.putCharSequence(ARG_FILE_NAME, fileName);

fragment.setArguments(args);
return fragment;
Expand All @@ -80,7 +83,16 @@ public View onCreateView(LayoutInflater inflater, ViewGroup container,
View view = inflater.inflate(R.layout.file_actions, null, false);
mListView = (ListView) view.findViewById(R.id.file_actions_list);

getDialog().getWindow().requestFeature(Window.FEATURE_NO_TITLE);
CharSequence title = getArguments().getCharSequence(ARG_FILE_NAME);
if(title != null && title.length() > 0) {
TextView header = (TextView)view.findViewById(R.id.file_actions_header);
header.setText(title);
header.setVisibility(View.VISIBLE);
view.findViewById(R.id.file_actions_header_divider).setVisibility(View.VISIBLE);
} else {
getDialog().getWindow().requestFeature(Window.FEATURE_NO_TITLE);
}

return view;
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -193,7 +193,7 @@ private void showFileAction(int fileIndex) {
}
}

FileActionsDialogFragment dialog = FileActionsDialogFragment.newInstance(menu, fileIndex);
FileActionsDialogFragment dialog = FileActionsDialogFragment.newInstance(menu, fileIndex, targetFile.getFileName());
dialog.setTargetFragment(this, 0);
dialog.show(getFragmentManager(), FileActionsDialogFragment.FTAG_FILE_ACTIONS);
}
Expand Down

0 comments on commit 7182b1d

Please sign in to comment.