RecyclerView scrolls to the top itself

Question

Hi I have a RecyclerView which I use to display images in two columns. ArrayList of movies holds 60 String of URL's for images.So the images are loaded fine into the ImageViews. When I scroll down everything works fine and smooth, then I reach bottom and scroll up, for a while it scrolls ok, but then it scrolls to the very top and I actually see all the photos between trying to load. I looked for this issue, but haven't found a cause of this scrolling issue. But if I have just 20 items in the list everything works great

Here's the Adapter's code:

public class CustomAdapter extends RecyclerView.Adapter<CustomAdapter.ViewHolder> { private ArrayList<Movie> movies; private Context context; private int newWidth; private int newHeight; public CustomAdapter(ArrayList<Movie> movies, Context context) { this.movies = movies; this.context = context; int screenWidth = context.getResources().getDisplayMetrics().widthPixels; int orientation = context.getResources().getConfiguration().orientation; newWidth = screenWidth / (orientation == Configuration.ORIENTATION_LANDSCAPE ? 3 : 2); newHeight = (int) (((double) newWidth / 185) * 278); } @Override public ViewHolder onCreateViewHolder(ViewGroup parent, int viewType) { View view = LayoutInflater.from(parent.getContext()).inflate(R.layout.item_movie, parent, false); return new ViewHolder(view); } @Override public void onBindViewHolder(ViewHolder holder, int position) { Picasso.with(context) .load(movies.get(position).url) .resize(newWidth, newHeight) .into(holder.imageView); } @Override public int getItemCount() { return movies.size(); } public static class ViewHolder extends RecyclerView.ViewHolder { final ImageView imageView; public ViewHolder(View itemView) { super(itemView); imageView = (ImageView) itemView.findViewById(R.id.pic); } } }

Thank you

I had the same issue but after adding tag to Picasso it worked.Just try as this way: Picasso.with(context) .load(IMAGE_BASE + W_185 + movies.get(position).image) .resize(newWidth, newHeight) .tag(context) .into(holder.image); — Tharindu Welagedara
– Tharindu Welagedara, Commented Nov 8, 2015 at 0:28
Please can you try different library like Glide(Google also recommend to use Glide). See this url inthecheesefactory.com/blog/… and I think "Disk Caching" section in that post will solve your problem. — Tharindu Welagedara
– Tharindu Welagedara, Commented Nov 8, 2015 at 0:52
@TdSoft yeap w/ Glide looks much better, still have some, but looks like Image Loader library is the issue. Thanks on that — Ihor Klimov
– Ihor Klimov, Commented Nov 8, 2015 at 1:07
Did you try disk caching strategy in Glide(.diskCacheStrategy(DiskCacheStrategy.ALL) — Tharindu Welagedara
– Tharindu Welagedara, Commented Nov 8, 2015 at 1:10

android developer · Accepted Answer · 2020-05-27 10:10:04Z

7

For me, I've noticed that if you use "match_parent" for some Views (ViewPager with RecyclerViews in my case) instead of constraints for ConstraintsLayout, this issue can occur.

I've reported it here.

For now, my workaround/fix was to set constraints.

answered May 27, 2020 at 10:10

android developer

117k162 gold badges794 silver badges1.3k bronze badges

Sign up to request clarification or add additional context in comments.

6 Comments

Mahmoud Mabrok Over a year ago

I am facing same issue but also constraints not solve the problem

android developer Over a year ago

@MahmoudMabrok You can create a POC and share it on the issue I've reported, then, or create a new one in case it's too different.

Mahmoud Mabrok Over a year ago

problem was using as i think now (i was using SwipRefresh after i delete it app works fine )

android developer Over a year ago

@MahmoudMabrok I don't understand. Again, if you can reproduce it on POC, report it.

Steven B. Over a year ago

Omg thanks, setting height on my ViewPager2 to 0dp and setting constraints fixed the issue. spent weeks breaking my head over this.

|

Ihor Klimov · Accepted Answer · 2015-11-14 20:26:40Z

I've found the solution.The problem was that my ImageView didn't have a fixed size, it got it when picture is being downloaded, that's why when I scrolled to top - there happened to be all of the previous ImageViews in one screen with 0 height and without picture, but when the picture is being dowloaded the ImageView size extended and looks like the RecyclerView scrolls to the first of the visible ImageView.

So I added this code to my CustomAdapter

@Override public ViewHolder onCreateViewHolder(ViewGroup parent, int viewType) { View view = LayoutInflater.from(parent.getContext()).inflate(R.layout.item_movie, parent, false); ImageView viewById = (ImageView) view.findViewById(R.id.poster); viewById.setMinimumWidth(newWidth); viewById.setMinimumHeight(newHeight); return new ViewHolder(view); }

this is how I change my ImageView size to the one I want before the picture actually being downloaded.

Also added a ProgressBar to look better.

@Override public void onBindViewHolder(final ViewHolder holder, int position) { holder.progressBar.setVisibility(View.VISIBLE); Picasso.with(context) .load(movies.get(position).postersUrl) .resize(0, newHeight) .noFade() .into(holder.imageView, new Callback() { @Override public void onSuccess() { holder.progressBar.setVisibility(View.INVISIBLE); } @Override public void onError() { } });

Collectives™ on Stack Overflow

RecyclerView scrolls to the top itself

2 Answers 2

6 Comments

Comments

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

6 Comments

Comments

Related