Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#3558 closed defect (fixed)

A flip horizontal issue

Reported by: massimo ceraolo Owned by: Adeel Asghar
Priority: high Milestone: 1.9.4
Component: OMEdit Version:
Keywords: Cc: davidluyando@…

Description

The enclosed "PULSER" model was created in Dymola.
AFAIK it is correct.
When used in another model, che OMEdit command "flip horizontal" does not flip the text, that is, I think, incorrect: output-related text will appear near the input connectors, and vice-versa.

Attachments (1)

PULSER.mo (2.1 KB ) - added by massimo ceraolo 9 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 by Francesco Casella, 9 years ago

Cc: davidluyando@… added
Milestone: Future1.9.4
omc_version: 1.9.4-dev-490
Priority: lowhigh

comment:2 by Francesco Casella, 9 years ago

I guess the problem is that when a component is flipped, the text boxes themselves should not be flipped (otherwise the text can become unreadable), but their position should, with respect to the baricenter of the icon.

In fact, the problem shows up already with basic models from the MSL. Just drag a voltage source from the Modelica.Electrical.Analog.Sources library, flip it horizontally, et voilà, the plus and minus sign do not swap place. This is very confusing, because the polarity of the generator as understood from the icon is wrong!

comment:3 by massimo ceraolo, 9 years ago

Today I discovered that flipping vertically already works better than flipping horizontally.

When vertically flipping Horizontal text is correctly flipped.
Vertical text, when vertical flipping, should be left unchanged, IMO. OMEdit rotates it by 180° (See enclosed model PULSER.mo).

Finally I've noted that to rotate icons OMEdit uses the same keyboard short-cuts as Dymola.
Therefore I would suggest to add in OMEdit short-cuts for flipping icons, and to choose the same short-cuts used by Dymola.

Fixing horizontal and vertical flip would make OMEdit really nicer. Are any plans to do this in the near future?

Last edited 9 years ago by massimo ceraolo (previous) (diff)

by massimo ceraolo, 9 years ago

Attachment: PULSER.mo added

comment:4 by massimo ceraolo, 9 years ago

Is it programmed to fix this ticket before the official 1.9.4 is issued?
It would be nice for me.

comment:5 by Adeel Asghar, 9 years ago

I will try to fix it next week.

comment:6 by Adeel Asghar, 9 years ago

Resolution: fixed
Status: newclosed

Fixed in 62c0c86/OMEdit.

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

Milestone: 1.9.41.9.4-1.9.x

Milestone renamed

comment:8 by Martin Sjölund, 9 years ago

Milestone: 1.9.4-1.9.x1.9.4

Milestone renamed

Note: See TracTickets for help on using tickets.