Phoenix icon indicating copy to clipboard operation
Phoenix copied to clipboard

Initialization problem in wx.lib.mixins.listctr

Open da-dada opened this issue 2 years ago • 0 comments

Windows: wxPython version & source: Python version & source:

current indicator should not be initialized to valid value:

lines 476 / 477 should be (click to expand)
        self.curRow = -1
        self.curCol = -1

and line 539 should then read

        if row < 0 or row != self.curRow:

there should also be a

        self.Bind(wx.EVT_LIST_ITEM_DESELECTED, self.OnItemDeselected, self)

with

   def OnItemDeselected(self, evt):
        self.curRow = -1
        evt.Skip()

and replace

        self.editor.Bind(wx.EVT_KILL_FOCUS, self.CloseEditor)

by

        def evt_kill_focus(evt):
            self.CloseEditor()
            evt.Skip()
        self.editor.Bind(wx.EVT_KILL_FOCUS, evt_kill_focus)

and move

                self.editor.Hide()

of method 'CloseEditor' to the last line of that method and delete

                self.SetFocus()

in that method altogether

da-dada avatar Nov 08 '23 16:11 da-dada