Commit ef2bee9c9c1b120caff2ec0eced5efdd3c992e0c
1 parent
81552f9d
Exists in
master
Doing nothing if clicked outside of a item in listctrl
Showing
1 changed file
with
13 additions
and
11 deletions
Show diff stats
invesalius/gui/data_notebook.py
@@ -386,22 +386,24 @@ class InvListCtrl(wx.ListCtrl): | @@ -386,22 +386,24 @@ class InvListCtrl(wx.ListCtrl): | ||
386 | def OnClickItem(self, evt): | 386 | def OnClickItem(self, evt): |
387 | self._click_check = False | 387 | self._click_check = False |
388 | item_idx, flag = (self.HitTest(evt.GetPosition())) | 388 | item_idx, flag = (self.HitTest(evt.GetPosition())) |
389 | - column_clicked = self.get_column_clicked(evt.GetPosition()) | ||
390 | - if column_clicked == 0: | ||
391 | - self._click_check = True | ||
392 | - item = self.GetItem(item_idx, 0) | ||
393 | - flag = not bool(item.GetImage()) | ||
394 | - self.SetItemImage(item_idx, int(flag)) | ||
395 | - self.OnCheckItem(item_idx, flag) | 389 | + if item_idx > -1: |
390 | + column_clicked = self.get_column_clicked(evt.GetPosition()) | ||
391 | + if column_clicked == 0: | ||
392 | + self._click_check = True | ||
393 | + item = self.GetItem(item_idx, 0) | ||
394 | + flag = not bool(item.GetImage()) | ||
395 | + self.SetItemImage(item_idx, int(flag)) | ||
396 | + self.OnCheckItem(item_idx, flag) | ||
396 | evt.Skip() | 397 | evt.Skip() |
397 | 398 | ||
398 | def OnDblClickItem(self, evt): | 399 | def OnDblClickItem(self, evt): |
399 | self._click_check = False | 400 | self._click_check = False |
400 | item_idx, flag = (self.HitTest(evt.GetPosition())) | 401 | item_idx, flag = (self.HitTest(evt.GetPosition())) |
401 | - column_clicked = self.get_column_clicked(evt.GetPosition()) | ||
402 | - if column_clicked == 1: | ||
403 | - item = self.GetItem(item_idx, 1) | ||
404 | - self.enter_edition(item) | 402 | + if item_idx > -1: |
403 | + column_clicked = self.get_column_clicked(evt.GetPosition()) | ||
404 | + if column_clicked == 1: | ||
405 | + item = self.GetItem(item_idx, 1) | ||
406 | + self.enter_edition(item) | ||
405 | evt.Skip() | 407 | evt.Skip() |
406 | 408 | ||
407 | def enter_edition(self, item): | 409 | def enter_edition(self, item): |