Skip to content
This repository has been archived by the owner on Jul 8, 2023. It is now read-only.

Wrong layout for Alt nested in Creation #406

Open
MrCoder opened this issue Nov 22, 2022 · 2 comments
Open

Wrong layout for Alt nested in Creation #406

MrCoder opened this issue Nov 22, 2022 · 2 comments
Labels
bug Something isn't working

Comments

@MrCoder
Copy link
Contributor

MrCoder commented Nov 22, 2022

new A(x) {
  if (condition){
    B.method()
  }
}

It also has impact on other fragment components:

  1. Loop
  2. Par
  3. Opt
  4. TCF (try/catch/finally)

It does not have impact on:

  1. simple message (sync and async)
  2. creation
  3. return/reply
@MrCoder MrCoder added the bug Something isn't working label Nov 22, 2022
@MrCoder
Copy link
Contributor Author

MrCoder commented Nov 23, 2022

Being a defect in the core, it has impact on all products:

  1. Confluence;
  2. JetBrains IDE plugins
  3. Chrome Extension;
  4. Web App;
  5. ZenUML.cn.

This is a medium to low bug as it only impact in a specific combination: Fragment under Creation. I would assume less than 10% diagrams use Creation, under which less than 5% will use a Fragment.

So we will fix it in the core (though there are quick fixes in each products).

@MrCoder
Copy link
Contributor Author

MrCoder commented Nov 23, 2022

So we will fix it in the core (though there are quick fixes in each products).

It turns out the proper fix is very complex on Confluence because confluence use a deeply coupled store. Let's fix it with a quick CSS fix on Confluence. It is not a high priority to fix it but since we are releasing other changes (generic viewer), let's fix it in the same release.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant