diff --git a/app/src/main/java/eu/faircode/email/FragmentDialogSearch.java b/app/src/main/java/eu/faircode/email/FragmentDialogSearch.java index 3a8a4cdeb3..618d128d1d 100644 --- a/app/src/main/java/eu/faircode/email/FragmentDialogSearch.java +++ b/app/src/main/java/eu/faircode/email/FragmentDialogSearch.java @@ -106,14 +106,14 @@ public class FragmentDialogSearch extends FragmentDialogBase { final CheckBox cbNotes = dview.findViewById(R.id.cbNotes); final CheckBox cbHeaders = dview.findViewById(R.id.cbHeaders); final CheckBox cbHtml = dview.findViewById(R.id.cbHtml); + final CheckBox cbSearchTrash = dview.findViewById(R.id.cbSearchTrash); + final CheckBox cbSearchJunk = dview.findViewById(R.id.cbSearchJunk); final CheckBox cbUnseen = dview.findViewById(R.id.cbUnseen); final CheckBox cbFlagged = dview.findViewById(R.id.cbFlagged); final CheckBox cbHidden = dview.findViewById(R.id.cbHidden); final CheckBox cbEncrypted = dview.findViewById(R.id.cbEncrypted); final CheckBox cbAttachments = dview.findViewById(R.id.cbAttachments); final Spinner spMessageSize = dview.findViewById(R.id.spMessageSize); - final CheckBox cbSearchTrash = dview.findViewById(R.id.cbSearchTrash); - final CheckBox cbSearchJunk = dview.findViewById(R.id.cbSearchJunk); final Button btnBefore = dview.findViewById(R.id.btnBefore); final Button btnAfter = dview.findViewById(R.id.btnAfter); final TextView tvBefore = dview.findViewById(R.id.tvBefore); @@ -242,16 +242,9 @@ public class FragmentDialogSearch extends FragmentDialogBase { cbSearchIndex.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() { @Override public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { - cbSenders.setEnabled(!isChecked); - cbRecipients.setEnabled(!isChecked); - cbSubject.setEnabled(!isChecked); - cbKeywords.setEnabled(!isChecked); - cbMessage.setEnabled(!isChecked); cbNotes.setEnabled(!isChecked); cbHeaders.setEnabled(!isChecked); cbHtml.setEnabled(!isChecked); - cbUnseen.setEnabled(!isChecked); - cbFlagged.setEnabled(!isChecked); cbHidden.setEnabled(!isChecked); cbEncrypted.setEnabled(!isChecked); cbAttachments.setEnabled(!isChecked); @@ -301,6 +294,20 @@ public class FragmentDialogSearch extends FragmentDialogBase { } }); + cbSearchTrash.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() { + @Override + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { + prefs.edit().putBoolean("last_search_trash", isChecked).apply(); + } + }); + + cbSearchJunk.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() { + @Override + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { + prefs.edit().putBoolean("last_search_junk", isChecked).apply(); + } + }); + spMessageSize.setOnItemSelectedListener(new AdapterView.OnItemSelectedListener() { @Override public void onItemSelected(AdapterView parent, View view, int position, long id) { @@ -318,20 +325,6 @@ public class FragmentDialogSearch extends FragmentDialogBase { } }); - cbSearchTrash.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() { - @Override - public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { - prefs.edit().putBoolean("last_search_trash", isChecked).apply(); - } - }); - - cbSearchJunk.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() { - @Override - public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { - prefs.edit().putBoolean("last_search_junk", isChecked).apply(); - } - }); - btnAfter.setOnClickListener(new View.OnClickListener() { @Override public void onClick(View v) { diff --git a/app/src/main/res/layout/dialog_search.xml b/app/src/main/res/layout/dialog_search.xml index 7298d4f63f..9c90159df7 100644 --- a/app/src/main/res/layout/dialog_search.xml +++ b/app/src/main/res/layout/dialog_search.xml @@ -251,7 +251,6 @@ android:id="@+id/cbSenders" android:layout_width="0dp" android:layout_height="wrap_content" - android:layout_marginStart="24dp" android:layout_marginTop="12dp" android:checked="true" android:text="@string/title_search_in_senders" @@ -264,7 +263,6 @@ android:id="@+id/cbRecipients" android:layout_width="0dp" android:layout_height="wrap_content" - android:layout_marginStart="24dp" android:layout_marginTop="12dp" android:checked="true" android:text="@string/title_search_in_recipients" @@ -277,7 +275,6 @@ android:id="@+id/cbSubject" android:layout_width="0dp" android:layout_height="wrap_content" - android:layout_marginStart="24dp" android:layout_marginTop="12dp" android:checked="true" android:text="@string/title_search_in_subject" @@ -290,7 +287,6 @@ android:id="@+id/cbKeywords" android:layout_width="0dp" android:layout_height="wrap_content" - android:layout_marginStart="24dp" android:layout_marginTop="12dp" android:text="@string/title_search_in_keywords" android:textAppearance="@style/TextAppearance.AppCompat.Medium" @@ -302,7 +298,6 @@ android:id="@+id/cbMessage" android:layout_width="0dp" android:layout_height="wrap_content" - android:layout_marginStart="24dp" android:layout_marginTop="12dp" android:checked="true" android:text="@string/title_search_in_message" @@ -315,7 +310,6 @@ android:id="@+id/tvSearchTextHint" android:layout_width="0dp" android:layout_height="wrap_content" - android:layout_marginStart="24dp" android:layout_marginTop="6dp" android:text="@string/title_search_text_hint" android:textAppearance="@style/TextAppearance.AppCompat.Small" @@ -328,7 +322,6 @@ android:id="@+id/cbNotes" android:layout_width="0dp" android:layout_height="wrap_content" - android:layout_marginStart="24dp" android:layout_marginTop="12dp" android:checked="true" android:text="@string/title_search_in_notes" @@ -341,7 +334,6 @@ android:id="@+id/cbHeaders" android:layout_width="0dp" android:layout_height="wrap_content" - android:layout_marginStart="24dp" android:layout_marginTop="12dp" android:text="@string/title_search_in_headers" android:textAppearance="@style/TextAppearance.AppCompat.Medium" @@ -353,7 +345,6 @@ android:id="@+id/cbHtml" android:layout_width="0dp" android:layout_height="wrap_content" - android:layout_marginStart="24dp" android:layout_marginTop="12dp" android:text="@string/title_search_in_html" android:textAppearance="@style/TextAppearance.AppCompat.Medium" @@ -361,22 +352,44 @@ app:layout_constraintStart_toStartOf="parent" app:layout_constraintTop_toBottomOf="@id/cbHeaders" /> + + + + + app:layout_constraintTop_toBottomOf="@id/cbSearchJunk" /> - - - -