wbrawner/thunderbird-android
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix selection offset and cleanup

Browse source 
When selecting or opening a message (not a thread) there was an offset
of 1 selecting the message above. When selecting the first message, it
crashed. Now this seems to be fixed.
This commit is contained in:
sedrubal 2016-11-20 17:41:35 +01:00
parent 08daf2dffb
commit dab9f39ff6
No known key found for this signature in database
GPG key ID: B83D4C8CEB8ABFF6
1 changed files with 5 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
k9mail/src/main/java/com/fsck/k9/fragment/MessageListFragment.java
View file

@ -15,7 +15,6 @@ import java.util.Map.Entry;
import java.util.Set;
import java.util.concurrent.Future;
import android.annotation.SuppressLint;
import android.app.Activity;
import android.app.DialogFragment;
import android.app.Fragment;
@ -673,7 +672,7 @@ public class MessageListFragment extends Fragment implements OnItemClickListener
View view = inflater.inflate(R.layout.message_list_fragment, container, false);
initializePullToRefresh(inflater, view);
initializePullToRefresh(view);
initializeLayout();
mListView.setVerticalFadingEdgeEnabled(false);
@ -943,7 +942,7 @@ public class MessageListFragment extends Fragment implements OnItemClickListener
}
}
private void initializePullToRefresh(LayoutInflater inflater, View layout) {
private void initializePullToRefresh(View layout) {
mSwipeRefreshLayout = (SwipeRefreshLayout) layout.findViewById(R.id.swiperefresh);
mListView = (ListView) layout.findViewById(R.id.message_list);
@ -1513,8 +1512,8 @@ public class MessageListFragment extends Fragment implements OnItemClickListener
}
private int listViewToAdapterPosition(int position) {
if (position > 0 && position <= mAdapter.getCount()) {
return position - 1;
if (position >= 0 && position < mAdapter.getCount()) {
return position;
}
return AdapterView.INVALID_POSITION;
@ -1522,7 +1521,7 @@ public class MessageListFragment extends Fragment implements OnItemClickListener
private int adapterToListViewPosition(int position) {
if (position >= 0 && position < mAdapter.getCount()) {
return position + 1;
return position;
}
return AdapterView.INVALID_POSITION;