Gulivertx / cbatmo

A Netatmo Weather Station Web-APP for Raspberry Pi & official Raspberry touchscreen

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Scrollbar in Rasberry Pi after 2.30 upgrade

jukkaforss opened this issue · comments

Seems that screen is bit bigger than 800x600 in 2.30 version. How I can fit screen again?

Will be fixed soon in version 2.3.1.

commented

Hello,

since we have any rain today i have this fidget scrollbars on my screen.
I have V2.3.1 installed.

20200602_194910_Trim.zip

Ho damn ! The scrollbars appears only when Rain graph as some data to render ? When there was no Rain there is no scrollbar ?
Also, your app is really shaking or is it coming from the video ?

Edit : could you please give me what web browser do you use and which version.

commented

Hello,

its really shaking.
And i think it comes from the rain.
If we have no rain then the data in 0 - 1 digit.
If we have rain or no data (NAN) then we have 3 digit (0,6 or NAN).
And then we have this scrollbars and shaking.

I use Chromium V 78.0.3904.108

20200603_094222_resized

I also use Chromium V 78.0.3904.108.
When this happening, if I killall chromium and restart it, it will be fine for awhile.
Here is video.
IMG_7375.zip

Ok thanks for your reports, I don’t have this trouble but I use webkit-gtk.
I also see on your screenshots that the padding for lot of elements are not correct with chromium. I’m gonna investigate this...

OK found the problem about the rain graph, the responsive chart is not re-render if the right values change. I will implement a render refresh of the graph each time the rain values change to force the graph re-render with the available width which should fix this shaking issue.

OK found the problem about the rain graph, the responsive chart is not re-render if the right values change. I will implement a render refresh of the graph each time the rain values change to force the graph re-render with the available width which should fix this shaking issue.

Can you explain, and give some hints, how to re-render the rain-graphic?

@tdostler to force React to re-render any components there are different ways, the most common is to change a value in the component state or props. But there is also the ability to add a key to a component and by updating the key value React will re-render the component (not re-render for this case but unmount and mount again the component), the last method is to use forceUpdate method (https://reactjs.org/docs/react-component.html#forceupdate).

I already tried to do it in the branch deploy_script, by adding a key on the graph component where the key has value of the rain module. So when the rain values change, the key of the graph get this new values, then the graph is unmount and mount again, but unfortunately it is not working, here is the change of my code : fc7beb8#diff-57f2be8e0124c6dfd7519e16ebadfed1R22-R28
I think I added the key to the bad component because it's on the chart itself, which is re-render correctly but do not know his available width, I will try to re-render the parent component of the chart which embed the graph and the values and see if it make any improvement...

Damn it!!!!
Still shaking. I’m gonna make an update this weekend for I hope definitively suppress this issue.
Related issue : #41
Thank’s @karba71 for the report