Opened 10 years ago

Closed 10 years ago

Last modified 7 years ago

#3362 closed enhancement (fixed)

Context menu names in OMEdit

Reported by: anonymous Owned by: Adeel Asghar
Priority: low Milestone: 1.9.4
Component: OMEdit Version: trunk
Keywords: Cc:

Description

In OMEdit, there is a Libraries Browser context menu item for unload files files named "Unload"; the icon shows a red cross.

When right-clicking on a model inside a package, there is still a command named "unload" having as icon a red cross, but this does a totally different thing: it deletes the model from the package it belongs to. Having the same name for different task is misleading.

I think it is better to use for this second function the name "delete".

Change History (5)

comment:1 by Adeel Asghar, 10 years ago

In both cases the same thing is done. It just unloads the model from the Compiler AST. There are no two different things.

comment:2 by anonymous, 10 years ago

From the programmer point of view it is the same thing; from the user point of view, I'm not sure.
I say this because when, later, the user saves the package, the model is not saved and the final result is that it is deleted from the package.

Maybe it is not an important question and no modification is needed. For me this ticket can be closed.

comment:3 by Adeel Asghar, 10 years ago

Resolution: fixed
Status: newclosed

I have given it a second thought and I guess you are right. I also looked at other Modelica tool e.g Dymola and it also use Delete for nested classes.

Fixed in e3397c/OMEdit

comment:4 by Dietmar Winkler, 9 years ago

Milestone: Futurepre1.9.4

It doesn't make sense to keep closed ticket in the "Future" milestone that were simply forgotten to assign to the correct milestone in the past.

comment:5 by Martin Sjölund, 7 years ago

Milestone: pre1.9.41.9.4

Removing the pre1.9.4 milestone in favor of 1.9.4.

Note: See TracTickets for help on using tickets.