We don't currently have a good way of allowing modules to add new
details panels, but we know it's needed.  How were you using
ConfigLoader.loadTabs before?  Were using using a custom argo.ini file
or some other mechanism to change its behavior?

On Nov 26, 2007 11:45 AM, Sérgio Lopes <[EMAIL PROTECTED]> wrote:

> (1)
> I've missed the thread. I've read it and it was not very clear to me if
> profiles need that the Modules subsystem to be initialized before the GUI.
> Can someone confirm this?

The profile subsystem wants to allow modules to be able to contribute
profiles, I think, but I'm not sure that this should require a
particular initialization order.  In general, there's too much
unnecessary coupling between different parts of the system making
things like this very brittle.  Everyone wants to be first (or last),
making it difficult to keep them all happy.

> (2)
> Anyway I have found a very simple solution. I moved the creation of ToDoPane
> from Main#initializeGUI(...) to ProjectBrowser#createPanels(...). It seems
> to me that this is a clean solution since I don't understand why the
> ToDoPane was created differently from the other panes.
> Any input on this?

This was split up to allow ArgoEclipse to reuse the same top level
panels (Explorer, ToDo, Details, Diagram editor) that the
ProjectBrowser creates, but in the Eclipse framework instead of an
ArgoUML main window.  I'd have to go back and look to see why it was
done the way it was, but it almost certainly was a compromise between
the "right" way and a way which would minimize the impact on ArgoUML.

Any changes to this need to work with both ArgoUML and ArgoEclipse.
I'll have a look to see what can be done to improve the current
structure.

> If this is ok, then I can prepare a consistent solution by removing the
> unused toDoPane argument from ProjectBrowser constructor and post it here.
>
> If both (1) and (2) are ok, than we have managed to make the order of
> initialization of Modules and GUI subsystems independent, even with
> Profiles.  :)

I'll see if this works with ArgoEclipse.  I don't remember off the top
of my head if there's a reason it won't.

Tom

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to