[Windows] Action attribute duplicated while scrolling

[Juho Vepsäläinen]

1. Set up an action list so that a scrollbar appears to the window. (Just adding one Blender action is enough.)
2. Select some attribute.
3. Scroll (move the scrollbar up/down directly or either using a mouse scroll. Make sure that the attribute is ~active~ (ie. editable atm)). -> Note how the attribute stays in the same location.

I tested this on Win7 (Professional) using wxPython 2.8.10.1 (x64-unicode).

[Stani (stani)]

Committed revision 1905.

$ bzr diff
=== modified file 'phatch/lib/pyWx/treeEdit.py'
--- phatch/lib/pyWx/treeEdit.py 2010-03-07 00:36:46 +0000
+++ phatch/lib/pyWx/treeEdit.py 2010-03-26 19:43:26 +0000
@@ -451,6 +451,7 @@
         self.popup.Show()
         self.resize_popup()
         if not isinstance(field, formField.FontFileField):
+ self.Bind(wx.EVT_SCROLLWIN, self.close_popup)
             wx.GetTopLevelParent(self).Bind(wx.EVT_LEAVE_WINDOW,
                 self.close_popup)
             self.evt_leave_window = True
@@ -463,7 +464,8 @@

     def close_popup(self, event=None):
         frame = wx.GetTopLevelParent(self)
- if frame and self.evt_leave_window:
+ if self and frame and self.evt_leave_window:
+ self.Unbind(wx.EVT_SCROLLWIN)
             frame.Unbind(wx.EVT_LEAVE_WINDOW)
             self.evt_leave_window = False
         if self.popup:
@@ -471,6 +473,8 @@
             self.set_form_field_value(self.popup_item, value_as_string)
             # self.update_form_relevance(self.popup_item)
         self.popup = self.popup_item = None
+ if event:
+ event.Skip()

 # # This would be logical but only works in wxPython2.6
 # # def get_popup_pos_offset_size(self, item):