add workaround for listview jumping on image re/load

This commit is contained in:
Andrew Dolgov 2017-06-05 18:19:08 +03:00
parent da7ef18ab6
commit 2732790025
2 changed files with 23 additions and 1 deletions

View File

@ -46,6 +46,7 @@ import android.view.TextureView;
import android.view.View; import android.view.View;
import android.view.View.OnClickListener; import android.view.View.OnClickListener;
import android.view.ViewGroup; import android.view.ViewGroup;
import android.view.ViewTreeObserver;
import android.view.WindowManager; import android.view.WindowManager;
import android.view.animation.Animation; import android.view.animation.Animation;
import android.view.animation.AnimationUtils; import android.view.animation.AnimationUtils;
@ -776,6 +777,19 @@ public class HeadlinesFragment extends Fragment {
view = v; view = v;
view.getViewTreeObserver().addOnPreDrawListener(new ViewTreeObserver.OnPreDrawListener() {
@Override
public boolean onPreDraw() {
View flavorImage = view.findViewById(R.id.flavor_image);
if (flavorImage != null) {
article.flavorViewHeight = flavorImage.getMeasuredHeight();
}
return true;
}
});
titleView = (TextView)v.findViewById(R.id.title); titleView = (TextView)v.findViewById(R.id.title);
feedTitleView = (TextView)v.findViewById(R.id.feed_title); feedTitleView = (TextView)v.findViewById(R.id.feed_title);
@ -1164,7 +1178,7 @@ public class HeadlinesFragment extends Fragment {
}); });
} }
//Log.d(TAG, "IMG: " + article.flavorImageUri + " STREAM: " + article.flavorStreamUri); //Log.d(TAG, "IMG: " + article.flavorImageUri + " STREAM: " + article.flavorStreamUri + " H:" + article.flavorViewHeight);
//Log.d(TAG, "TAG:" + holder.flavorImageOverflow.getTag()); //Log.d(TAG, "TAG:" + holder.flavorImageOverflow.getTag());
holder.flavorImageView.setVisibility(View.VISIBLE); holder.flavorImageView.setVisibility(View.VISIBLE);
@ -1176,6 +1190,13 @@ public class HeadlinesFragment extends Fragment {
holder.flavorImageView.setMaxHeight((int)(m_screenHeight * 0.8f)); holder.flavorImageView.setMaxHeight((int)(m_screenHeight * 0.8f));
holder.flavorProgressTarget.setModel(article.flavorImageUri); holder.flavorProgressTarget.setModel(article.flavorImageUri);
if (article.flavorViewHeight > 0) {
RelativeLayout.LayoutParams lp = (RelativeLayout.LayoutParams) holder.flavorImageView.getLayoutParams();
lp.height = article.flavorViewHeight;
holder.flavorImageView.setLayoutParams(lp);
}
/* TODO: maybe an option? force height for all images to reduce list jumping around /* TODO: maybe an option? force height for all images to reduce list jumping around
RelativeLayout.LayoutParams lp = (RelativeLayout.LayoutParams) holder.flavorImageView.getLayoutParams(); RelativeLayout.LayoutParams lp = (RelativeLayout.LayoutParams) holder.flavorImageView.getLayoutParams();

View File

@ -45,6 +45,7 @@ public class Article implements Parcelable {
transient public String flavorStreamUri; transient public String flavorStreamUri;
transient public String youtubeVid; transient public String youtubeVid;
transient public List<Element> mediaList = new ArrayList<>(); transient public List<Element> mediaList = new ArrayList<>();
transient public int flavorViewHeight;
public Article(Parcel in) { public Article(Parcel in) {
readFromParcel(in); readFromParcel(in);