[Qt] new qt patches

Felipe Monteiro de Carvalho felipemonteiro.carvalho at gmail.com
Sat Nov 4 20:09:30 CET 2006


Thanks, I am having trouble applying your patch. How did you create
it? The prefered methods are "svn diff > myqtpatch.patch" or using
Create Patch command from TortoiseSVN .

This e-mail only arrived 8 hours because it was sent due to some
problems on the mailling list.

The following comments you put on the code:

+  // Sets it´ s initial properties ... do we really need it here ?
+  QWidget_setGeometry(Widget, AWinControl.Left, AWinControl.Top,
+   AWinControl.Width, AWinControl.Height);

Without this the initial position and size of the control will not be
set. Where else would you put this code?

+class procedure TQtWSScrollBar.SetParams(const AScrollBar: TCustomScrollBar);
+var
+  QtScrollBar: TQtScrollBar;
+begin
+  QtScrollBar := TQtScrollBar(AScrollBar.Handle);	
+// TODO: Check HeightForWidth() set here or we have lazarus bug ?
+// AScrollBar.Width;

I did not understand this comment. I also don´t know what
HeightForWidth is suppose to do =)

-- 
Felipe Monteiro de Carvalho



More information about the Qt mailing list