Changes between Initial Version and Version 3 of Ticket #453

Timestamp:

Jul 12, 2015 10:05:13 PM (9 years ago)

Author:

butler

Comment:

Indeed ticket #424 was closed too hastily. It turns out there were several problems that still needed to be addressed.

1. The main problem as identified in that ticket were EndEdit and PaintBackground having the wrong number of parameters. GridCellEditor was overwritten in data_processor and init and EndEdit methods were overwritten. However PaintBackground was not. While the error (too many arguments given) continued to be generated every keystroke it no longer prevented the cell from being edited. This was fixed by overriding the PaintBackground class, addding it to the new GridCellEditor class in data_processor (a bit tricker since it needed to call not the parent version but the parent's parent). No errors seem to be generated on editing any more.
2. A few of the Grid events being used were not bound to the local methods in the process of the cleanup. In particular EVT_GRID_LABEL_RIGHT_CLICK used to bring up the context menu that allows for insertion of a column and EVT_GRID_LABEL_LEFT_CLICK that is needed to properly set the data ranges for plotting. These have been added back calling the appropriate code.
3. In the process of removing the Edit_Menu item the on_open_menu method's section dealing with that menu item was not commented out. Thus any later menu would get the id such that when opened and that method called would execute the section written for the Edit menu and raise an exception. The removal of the menu item was done so as to be able to be able to easily add it back later if we choose to. Thus rather than commenting the section out, the self.edit = NONE was replaced wth an unused handle self.edit = wx.Menu() preventing the next menu item to get confused. This means that re-implementing the menu item should just be a question of commenting out the self.edit line and uncommenting the line beneath it which calls the add_edit_menu method. Meanwhile everything seems to now work without generating any errors without the menu being added.