Apart from the minor comments and Qt check I think it may also be useful ...
14 years, 9 months ago
(2009-07-07 04:59:36 UTC)
#3
Apart from the minor comments and Qt check I think it may also be useful to have
this on the toolbar for the Mayavi scene window? That would be convenient when
using this from mlab but this is only a wish and you may choose to not do it.
thanks.
prabhu
http://codereview.appspot.com/88108/diff/1/3
File enthought/mayavi/action/help.py (right):
http://codereview.appspot.com/88108/diff/1/3#newcode67
Line 67: TVTKClassChooser().edit_traits()
While this may work with wx. I have a feeling it may not work with Qt, can you
please check it and see. A long while back you needed to hold a reference to
the UI object that edit_traits returned for the UI to stay alive with Qt. I am
not sure that restriction was removed by Phil.
http://codereview.appspot.com/88108/diff/1/3#newcode91
Line 91: class TvtkClassBrowser(Action):
I would prefer this to be called TVTKClassBrowser since TVTK is an acronym.
http://codereview.appspot.com/88108/diff/1/2
File enthought/mayavi/plugins/mayavi_ui_action_set.py (right):
http://codereview.appspot.com/88108/diff/1/2#newcode167
Line 167: id = "TvtkClassBrowser",
Call the ID TVTKClassBrowser.
Issue 88108: Adding the TVTK class browser to the Mayavi help menu
(Closed)
Created 14 years, 9 months ago by Gael.Varoquaux
Modified 14 years, 9 months ago
Reviewers: prabhu
Base URL: https://svn.enthought.com/svn/enthought/Mayavi/trunk/
Comments: 3