Paul McNett wrote:
> Do you remember why we overrode Layout() to call update()? It seems that, if 
> anything, Layout() calling refresh() would be more appropriate.

You must have had a good reason, back in 2006 with [1839], which included this:

841   def Layout(self):
842     super(dForm, self).Layout()
843 -   wx.CallAfter(self.refresh)
844 +   wx.CallAfter(self.update)

But it could have been just-for-testing code that made it into the commit, 
because it wasn't mentioned in the log entry:

[EMAIL PROTECTED]:~/dabo/dabo/ui/uiwx$ svn log -vr 1839
------------------------------------------------------------------------
r1839 | ed | 2006-01-23 05:07:35 -0800 (Mon, 23 Jan 2006) | 9 lines
Changed paths:
   M /trunk/dabo/ui/uiwx/__init__.py
   M /trunk/dabo/ui/uiwx/dEditor.py
   M /trunk/dabo/ui/uiwx/dForm.py
   M /trunk/dabo/ui/uiwx/dMenu.py
   M /trunk/dabo/ui/uiwx/dPageFrameMixin.py
   M /trunk/dabo/ui/uiwx/uiApp.py

Fixed a quirk in how Windows handles menu events.

Added 'removePage()' and 'cyclePages()' functions to the dPageFrameMixin class. 
The former allow for a specific page to be removed, while the latter accepts an 
integer value and cycles the selected page forward (positive) or backwards 
(negative).

Added a clear() function to dMenu() for removing all current items.

Improved the default file name handling for dEditor.


------------------------------------------------------------------------

There were a ton of mostly cosmetic changes to dForm.py, so perhaps you 
overlooked this real change in the noise?

Paul


_______________________________________________
Post Messages to: [email protected]
Subscription Maintenance: http://leafe.com/mailman/listinfo/dabo-dev
Searchable Archives: http://leafe.com/archives/search/dabo-dev
This message: http://leafe.com/archives/byMID/[EMAIL PROTECTED]

Reply via email to