Skinparam from command line breaks salt diagrams

Question

Cross-posting form https://github.com/qjebbs/vscode-plantuml/issues/164 as I suspet this may be a PlantUML issue, not a VSCode plugin issue.

I have the following diagram:

@startuml

salt

{

Just plain text

[This is my button]

() Unchecked radio

(X) Checked radio

[] Unchecked box

[X] Checked box

"Enter text here "

^This is a droplist^

}

@enduml

I also use a skinparam on the command line to work around the component diagram rendering bug:

-SfixCircleLabelOverlapping=true

Upon trying to render the diagram, I get the following error:

https://i.imgur.com/h5wdbJJ.png

I suspect the skinparam command line argument and salt syntax are not working well together here. I need to use the same PlantUML command for all rendering, so I cannot just use a different one for salt diagrams.

Comments

Although I gave an answer based on the available code, I think an essential part in the example (the skinparmeter) is missing. Please specify the used skinparameter / commandline / ...

---

I have edited the post to include the command line. Apologies for forgetting it!

Note: the same errors occurs if I try to use "version" with the skinparameter command line!

---

With version it is logical. This is a special command and has to be the first command (as far as I know). A skinparameter with the version command does not make a lot of sense either (I think).

---

I agree the skinparam does not make sense for all diagrams, such as version and others.

Of course, I am not going to customize my build script to give a separate command line arguments for each diagram, so this is a bit of a problem if I want to have a generic solution for building diagrams.

---

From where dit you get the skinparameter: fixCircleLabelOverlapping as I ran plantulml with -language and in this list I don't see this skinparameter.

---

This should be fixed with last beta http://beta.plantuml.net/plantuml.jar

As @albert says, the skinparam does not make sense with salt. So we simply ignore it in that case.

This is an indirect side effect coming from -SfixCircleLabelOverlapping=true

We agree that customizing build script would be a pain. We have also corrected this for "version" but we did not test it widely.

Please post here if you find other issues and thanks for the feeback!

---

@plantuml has the list of skinparameters with -language also been updated? I still don't see it in the latest version (PlantUML version 1.2018.13beta3)

---

@plantuml when using the -SfixCircleLabelOverlapping=true with the version 1.2018.13beta3 I still get an error message:
    Error line 1 in file: D:\...\aa.pu
    Some diagram description contains errors

---

We did not test enough :-)
1.2018.13beta4 should be better http://beta.plantuml.net/plantuml.jar
skinparameters with -language has also been updated.

---

Thanks, the example now works and I see the skinparameter in the list.

---

Thank you for the fix @plantuml! I am very impressed by the speed of your responses!