JAM-Software / Virtual-TreeView

Virtual Treeview is a Delphi treeview control
http://www.jam-software.de/virtual-treeview/
646 stars 253 forks source link

Vertical Scroll Bar Cannot Scroll To Bottom In Select Circumstances #983

Open Joshua-Douglas opened 4 years ago

Joshua-Douglas commented 4 years ago

Hello,

I've come across a bug that is preventing users from scrolling to the bottom of a virtual string tree node list in select circumstances. I've attached a sample application that should reveal this issue with the following reproduction steps. This issue was seen with the most recent version of the code base available on GitHub. If need any help reproducing the issue, please let me know.

Reproduction Steps

Note the following:

VSTScrollingBug.zip

joachimmarder commented 4 years ago

Once the longest node is out of view you will note that 1) the horizontal scroll bar is still present, even though all of the nodes in view fit comfortably in the window

I cannot confirm this, tested in RAD Studio 10.3 with latest source. Please note even a 1 pixel line of the long node's row in the view is sufficient for the scrollbar to appear, even if this line does not have any text in it. grafik grafik

Joshua-Douglas commented 4 years ago

Hello Joachimmarder,

Thank you for responding so quickly.

I built the sample application on a separate machine with Delphi 10.3 and a fresh clone of the GitHub repo. Unfortunately, I'm still seeing the issue. In the second screenshot you provided, were you able to scroll down to the bottom node without the scroll bar automatically scrolling back up?

I made a 30-second video that shows what I am seeing. Could you take another look and let me know if you still are not seeing the issue shown in the video?

Thanks again for the time and effort, all help is appreciated.

Recording of VST Scroll Bug.zip

Best, Josh

joachimmarder commented 4 years ago

OK, the problem occurs if the newly created node appears behind the horizontal scrollbar. Editing the node is not needed. The problem disappears again if another node is created. The problem seems to be in TBaseVirtualTree.ScrollIntoView() in the else-part. I never visited this code myself.

MHumm commented 2 years ago

I encrountered the same thing. My workaround is to enable AlwaysVisible for the scrollbars.

timetraveler104 commented 1 year ago

I am encountering the same thing. Any updates on fix for this? This makes component unusable when larger amount of nodes are visible.

Best regards, Mladen

EvMac0 commented 1 year ago

Faced with the same error, fix it please!