Virtual TreeView pathc for VCL styles

86 views
Skip to first unread message

Joachim Marder

unread,
Nov 3, 2014, 4:24:18 AM11/3/14
to virtual-...@googlegroups.com, Dietmar Rösler, Roman Kassebaum
Hi fellows.

Yesterday I committed a larger patch that I received from Dietmar Rösler which addresses various issue regarding VCL styles (especially #478 and #491). It would be great if those of you who use VCL styles could test the trunk version of Virtual Treeview in your application to check if the patch has any unwanted side effects.

Thanks and best regards,

Joachim




Achim Kalwa

unread,
Nov 11, 2014, 3:49:22 AM11/11/14
to virtual-...@googlegroups.com, du...@netcologne.de

Hello Joachim,

Yesterday I committed a larger patch that I received from Dietmar Rösler which addresses various issue regarding VCL styles (especially #478 and #491). It would be great if those of you who use VCL styles could test the trunk version of Virtual Treeview in your application to check if the patch has any unwanted side effects.
 
Something in the trunk version (6.0.0?) makes the header painting wrong on Windows 7 (both Aero and Basic style). Branch V5.5.2 has the correct painting. Please see attached screenshot image. Left window is using VST V5.5.2, right image is from VST 6.0 (Trunk). I've also attached the small sample project.

There are no special VCL styles applied; just plain VCL Win32 application with Delphi XE7.

Regards,
Achim



VST V6 Header Bug.png
TestWin7.zip

Joachim Marder

unread,
Nov 11, 2014, 8:37:02 AM11/11/14
to virtual-...@googlegroups.com, Dietmar Rösler
Hello Achim.

I just applied a patch from Dietmar. Did the situation improve for you?

Best regards,

Joachim


--
You received this message because you are subscribed to the Google Groups "Virtual Treeview" group.
To unsubscribe from this group and stop receiving emails from it, send an email to virtual-treevi...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Achim Kalwa

unread,
Nov 11, 2014, 9:40:48 AM11/11/14
to virtual-...@googlegroups.com, du...@netcologne.de
Hello Joachim,

thank you for the quick response.

Am Dienstag, 11. November 2
 
I just applied a patch from Dietmar. Did the situation improve for you?

The columns itself are now painted correctly; however the remaining part of the header bar is still painted wrong (the "unused" area). Screenshot attached.

Achim
VST6 Header Painting.png

Joachim Marder

unread,
Nov 14, 2014, 6:47:38 AM11/14/14
to virtual-...@googlegroups.com, Dietmar Rösler
Hi Achim.

Would you mind testing the latest trunk version again? It now includes a further patch from Dietmar.

Thanks and best regards,

Joachim


--

Achim Kalwa

unread,
Nov 17, 2014, 7:00:39 AM11/17/14
to virtual-...@googlegroups.com, du...@netcologne.de
Hello Joachim,

sorry for the delay; I couldn't test over the weekend.
The version from trunk now paints the header correctly on Windows 7 Basic an Areo style; but when closing the application an exception is raised.

I've tracked down the bug to TVTHeader.Destroy; which causes TCollection.Changed() to be called, which in turn calls 
TVirtualTreeColumn.Update -> TVirtualTreeColumn.UpdatePositions -> TBaseVirtualTree.UpdateHorizontalScrollBar -> Boom.

I've inserted a line
  if csDestroying in FHeader.Treeview.ComponentState then Exit;
into TVirtualTreeColumns.UpdatePositions to fix this problem.

Achim

Joachim Marder

unread,
Nov 17, 2014, 9:45:08 AM11/17/14
to virtual-...@googlegroups.com, Dietmar Rösler
Thanks Achim, I just committed this change to the trunk.

--
Reply all
Reply to author
Forward
0 new messages