search

rainerf / ricodebug

ricodebug is a debugger featuring data visualization, similar to DDD
12 stars 9 forks source link

Add remove variable to watchwindow context menu #38

Closed giselher closed 12 years ago

giselher commented 12 years ago

This pull request adds the possibility to extend the context menu of the TreeItemView through overwriting the inherit method and still calling the function of the superclass.

Doing it this way makes it very easy to add more action and it is also very flexible.

The way the context menu works in the datagraph is too specific to generalise it. But I could try to unify it by writing something like helper/context_menu.py.

giselher commented 12 years ago

I can't reproduce the bug in the last comment. Could you please explain the steps you did to encounter the bug?

rainerf commented 12 years ago
  1. Add a struct or something else that has children.
  2. Right-click a child, you're presented with an option to remove it (see below) screenshot

This should not be possible. Clicking it, by the way, results in an exception.

giselher commented 12 years ago

Fixed in the last commit.