Layouting of lifelines broken

Issue #145 resolved
Matthias Schoettle created an issue

The layouting of lifelines has two problems:

  1. Lifelines that were woven don't have a LayoutElement associated. Either copy them during weaving to obtain the old location or create an "empty" one such that the position will be determined during layouting.
  2. When layouting lifelines, a lifeline is considered multiple times, meaning that the next x-position is increased too often.

Comments (4)

  1. Matthias Schoettle reporter

    Resolved issue #145: When creating a LifelineView, create a LayoutElement if it doesn't exist yet. This can happen when the lifeline was woven from an advice. This is not ideal, but the user can then re-align the lifelines as preferred.

    → <<cset 14dfdc0aa617>>

  2. Matthias Schoettle reporter
    • changed status to open

    The latest commit breaks creating a lifeline the regular way, because initially there is no layout (it will be added in the next command). So a layout element is created, which triggers a notification and causes a NPE in MessageViewView line 657.

    Actually, the code to create a layout element is already there in the build method.

  3. Log in to comment