Scott - beware that emailing a bug in the Debian BTS doesn't forwards
to the submitter by default (you can cc: nnnnnn-submit...@bugs.debian.org
to do that). I only saw your reply because I happened to be checking
bugs prior to raising the severity.
On Wed, Sep 10, 2014 at 11:24:21PM -0400, Scott Talbert wrote:
> Here's an updated debdiff with an updated patch for wxPython 3.0 support for
> congruity. I also added in a trivial fix for the other open bug too - I
> hope that is OK.
Thanks.
> A couple of lessons learned that you may want to include in your README for
> wxPython 3 porting (unfortunately I don't think these can be automated in
> your script):
>
> a) In a couple of places we were calling wx.Sizer.Remove(wx.Window). This
> worked in 2.8 but it seems it now does not. We should have instead be using
> wx.Sizer.Detach(wx.Window)
Ah yes, I've hit this with C++ porting. I think the change is because
for other types Remove() deletes so it's confusing that it doesn't for
wxWindow.
> b) SetSize() no longer does exactly the same thing that it did on 2.8. We
> had to call SetClientSize() instead to get the same behavior as we had in
> 2.8.
It was a documented in 2.9.0 as a change for wxGTK that:
| - wxTopLevelWindow::GetSize() returns the size including the WM decorations.
So I guess SetSize() may have changed correspondingly. That could explain
the odd case of a window appearing to be a little smaller than it should
be...
> c) wx.App(False) can be used in place of wx.PySimpleApp.
Aha - that's very useful to know. I couldn't find any documentation about
what was meant to replace wx.PySimpleApp.
I have updated the README with the above info.
Mathieu: We have a tested fix now, and one with upstream's blessing. Let
me know if you'd like me to NMU it.
Cheers,
Olly
--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
