diff --git a/app/src/main/java/eu/faircode/email/FragmentMessage.java b/app/src/main/java/eu/faircode/email/FragmentMessage.java index b2364fc5..5a0de68b 100644 --- a/app/src/main/java/eu/faircode/email/FragmentMessage.java +++ b/app/src/main/java/eu/faircode/email/FragmentMessage.java @@ -198,12 +198,12 @@ public class FragmentMessage extends FragmentEx { @Override public boolean onNavigationItemSelected(@NonNull MenuItem item) { switch (item.getItemId()) { - case R.id.action_trash: - onActionDelete(id); - return true; case R.id.action_spam: onActionSpam(id); return true; + case R.id.action_trash: + onActionDelete(id); + return true; case R.id.action_move: onActionMove(id); return true; @@ -333,8 +333,8 @@ public class FragmentMessage extends FragmentEx { top_navigation.getMenu().findItem(R.id.action_reply_all).setVisible(!outbox && message.cc != null); top_navigation.setVisibility(View.VISIBLE); - bottom_navigation.getMenu().findItem(R.id.action_trash).setVisible(!outbox && !trash && hasTrash); bottom_navigation.getMenu().findItem(R.id.action_spam).setVisible(!outbox && !junk && hasJunk); + bottom_navigation.getMenu().findItem(R.id.action_trash).setVisible(!outbox && !trash && hasTrash); bottom_navigation.getMenu().findItem(R.id.action_move).setVisible(!outbox && (!inbox || hasUser)); bottom_navigation.getMenu().findItem(R.id.action_archive).setVisible(!outbox && !archive && hasArchive); bottom_navigation.getMenu().findItem(R.id.action_reply).setVisible(!outbox); @@ -439,6 +439,35 @@ public class FragmentMessage extends FragmentEx { .putExtra("action", "reply_all")); } + private void onActionSpam(final long id) { + AlertDialog.Builder builder = new AlertDialog.Builder(getContext()); + builder + .setMessage(R.string.title_ask_spam) + .setPositiveButton(android.R.string.ok, new DialogInterface.OnClickListener() { + @Override + public void onClick(DialogInterface dialog, int which) { + executor.submit(new Runnable() { + @Override + public void run() { + try { + DB db = DB.getInstance(getContext()); + EntityMessage message = db.message().getMessage(id); + message.ui_hide = true; + db.message().updateMessage(message); + + EntityFolder spam = db.folder().getFolderByType(message.account, EntityFolder.TYPE_JUNK); + EntityOperation.queue(getContext(), message, EntityOperation.MOVE, spam.id); + EntityOperation.process(getContext()); + } catch (Throwable ex) { + Log.e(Helper.TAG, ex + "\n" + Log.getStackTraceString(ex)); + } + } + }); + } + }) + .setNegativeButton(android.R.string.cancel, null).show(); + } + private void onActionDelete(final long id) { String folderType = (String) bottom_navigation.getTag(); if (EntityFolder.TYPE_TRASH.equals(folderType)) { @@ -488,35 +517,6 @@ public class FragmentMessage extends FragmentEx { } } - private void onActionSpam(final long id) { - AlertDialog.Builder builder = new AlertDialog.Builder(getContext()); - builder - .setMessage(R.string.title_ask_spam) - .setPositiveButton(android.R.string.ok, new DialogInterface.OnClickListener() { - @Override - public void onClick(DialogInterface dialog, int which) { - executor.submit(new Runnable() { - @Override - public void run() { - try { - DB db = DB.getInstance(getContext()); - EntityMessage message = db.message().getMessage(id); - message.ui_hide = true; - db.message().updateMessage(message); - - EntityFolder spam = db.folder().getFolderByType(message.account, EntityFolder.TYPE_JUNK); - EntityOperation.queue(getContext(), message, EntityOperation.MOVE, spam.id); - EntityOperation.process(getContext()); - } catch (Throwable ex) { - Log.e(Helper.TAG, ex + "\n" + Log.getStackTraceString(ex)); - } - } - }); - } - }) - .setNegativeButton(android.R.string.cancel, null).show(); - } - private void onActionMove(final long id) { Bundle args = new Bundle(); args.putLong("id", id); diff --git a/app/src/main/res/menu/action_view_bottom.xml b/app/src/main/res/menu/action_view_bottom.xml index 45d7ef02..ce76b858 100644 --- a/app/src/main/res/menu/action_view_bottom.xml +++ b/app/src/main/res/menu/action_view_bottom.xml @@ -2,18 +2,18 @@