search

Ashinch / ReadYou

An Android RSS reader presented in Material You style.
GNU General Public License v3.0
4.87k stars 189 forks source link

Native components renderer scroll issue #517

Open kid1412621 opened 8 months ago

kid1412621 commented 8 months ago

1. Environment

2. Describe the bug

After enabled readability, fastly scroll down to bottom, then slowly scroll up a little. Unable to scroll up anymore.

https://github.com/Ashinch/ReadYou/assets/26278054/39e9b41a-441d-477d-b608-47caaa9186ab

mbestavros commented 8 months ago

@kid1412621 I'm unable to reproduce the bug on my device. Are you seeing it when app language is set to English too?

Ashinch commented 8 months ago

This is a definite bug. The problem occurs in large pieces of composable items such as pictures. When scrolling to the screen again, it is re-rendered. Due to the characteristics of compose, we can't measure the height of all the composable items in the whole article at the same time, so I guess this problem cannot be avoided when using composable items to render the article. Similar to the causes of this problem are: scroll bar jitter.

（I hope I can explain it clearly)

I will provide users choice to render articles with WebView later.

At that time, such issues will be closed.

kid1412621 commented 8 months ago

@kid1412621 I'm unable to reproduce the bug on my device. Are you seeing it when app language is set to English too?

Yes, the app lang is English. Maybe lots of images in article will help to reproduce.

This is a definite bug. The problem occurs in large pieces of composable items such as pictures. When scrolling to the screen again, it is re-rendered. Due to the characteristics of compose, we can't measure the height of all the composable items in the whole article at the same time, so I guess this problem cannot be avoided when using composable items to render the article. Similar to the causes of this problem are: scroll bar jitter.

（I hope I can explain it clearly)

I will provide users choice to render articles with WebView later.

At that time, such issues will be closed.

So this issue might happen even without Readability?

Ashinch commented 8 months ago

This is a definite bug. The problem occurs in large pieces of composable items such as pictures. When scrolling to the screen again, it is re-rendered. Due to the characteristics of compose, we can't measure the height of all the composable items in the whole article at the same time, so I guess this problem cannot be avoided when using composable items to render the article. Similar to the causes of this problem are: scroll bar jitter.

（I hope I can explain it clearly)

110

Ashinch commented 8 months ago

So this issue might happen even without Readability?

Yes.

I will use the webview renderer to bypass this problem.