wxperl-users

Re: [wxperl-users] Additions to Wx::Grid

[Mattia B. \<mattia\.barbon\@libero\.it\> <mat...@li...>]

  Hello,

> I have wrapped a few of the newer methods added to the Wx::Grid class:
> 
> Wx::Grid::EnableDragColMove
> Wx::Grid::GetCellOverflow
> Wx::Grid::GetDefaultCellOverflow
> Wx::Grid::SetDefaultCellOverflow

> I have not used #if WXPERL_W_VERSION_GE at all as I do not know when th=
ese
> methods were introduced, apart from EnableDragColMove which I believe w=
as
> just added now in 2.7.0.

  Looking into $WXWIDGETS/docs/changes.txt might help, as well as
http://cvs.wxwidgets.org/viewcvs.cgi/wxWidgets/include/wx 
 =3D> generic =3D> grid.h

> All tested and working on wxWidgets-MSW-2.7.0-rc1

  This patch will probably apply cleanly (I am not at home now to try);
a couple of suggestions: it is usually a good idea to attach patches as
well as inlining them (inlined =3D> quick to review, attached =3D> always=

safe from mail clients). I (and most open source projects I know) prefer
unified diffs instead of context diffs (the one you used). Context diff
is good, unified is better :-)

Thanks!
Mattia
=0A=0A=0A________________________________________________________________=
___=0ASalva ora il tuo preventivo Direct Line e assicurati lo sconto extr=
a 5+5% sulla polizza auto entro il 31 Luglio=0Ahttp://click.libero.it/dir=
ectline=0A

[wxperl-users] Addition to Sizer.xs

[Mark W. <ma...@ne...>]

Hello,

I have added Wx::SizerItem->IsShown() which was missing.

I hope this is the correct format.

Regards
Mark

Index: Sizer.xs
===================================================================
RCS file: /cvsroot/wxperl/wxPerl/XS/Sizer.xs,v
retrieving revision 1.46
diff -u -r1.46 Sizer.xs
--- Sizer.xs	18 Jun 2006 11:13:14 -0000	1.46
+++ Sizer.xs	26 Jul 2006 08:46:28 -0000
@@ -498,6 +498,9 @@
 wxSizerItem::IsWindow()
 
 bool
+wxSizerItem::IsShown()
+
+bool
 wxSizerItem::IsSizer()
 

Re: [wxperl-users] Addition to Sizer.xs

[Mattia B. <mat...@li...>]

On Wed, 26 Jul 2006 10:55:49 +0200
"Mark Wardell" <ma...@ne...> wrote:

> Hello,
> 
> I have added Wx::SizerItem->IsShown() which was missing.
> 
> I hope this is the correct format.

  Indeed it is.

Thanks, applied!
Mattia