[Octave-bug-tracker] [bug #53046] Variable Editor: feature request to make multiple tables visible at once.

[Dan Sebald]

Follow-up Comment #46, bug #53046 (project octave):

OK, this one is ready for review.  I've done some significant restructuring,
but I think you'll like it.  Now the various functions are associated with the
classes that are most relevant.  There is a fair amount of simplification and
removal of a layer of indirection as a result.  In summary, there are now the
following classes:


variable_editor
variable_dock_widget (now just a few routines related to focus)
[holds]
variable_editor_stack (a few routines that manages view)
[holds]
variable_editor_view (the bulk of routines moved from variable_editor)


and naturally there is the additional QTextWidget contained in
variable_editor_stack, but no real benefit to deriving a class in that case.

So, think of this as the majority of functionality being in the two classes

variable_editor_view
variable_editor_model

and then a few "glue" classes to organize things.  Patch attached.

(file #43368)
    _______________________________________________________

Additional Item Attachment:

File name: octave-multipanel_variable_editor-djs2018feb19.patch Size:101 KB


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?53046>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/