plantuml-stdlib / C4-PlantUML

C4-PlantUML combines the benefits of PlantUML and the C4 model for providing a simple way of describing and communicate software architectures
MIT License
6.41k stars 1.1k forks source link

$shadowing parameter of AddContainerTag no longer has any effect in current version of PlantUML Server (1.2022.4) #224

Closed dannief closed 2 years ago

dannief commented 2 years ago

See example below:

@startuml
!include https://raw.githubusercontent.com/plantuml-stdlib/C4-PlantUML/master/C4_Container.puml
!include <office/Servers/application_server>

AddContainerTag("webApp", $sprite="application_server", $legendText="web app container", $shadowing="true")

Container(web_app, "Web Application", "Java, Spring MVC, Tomcat 7.x", $tags="webApp")

SHOW_LEGEND()
@enduml

example

kirchsth commented 2 years ago

Hi @dannief

I entered a bug in forum. Typically it is fast fixed (In the last months they did a lot of style changes/improvements)

I will inform you as soon it is fixed.

BR Helmut

kirchsth commented 2 years ago

Hi @dannief,

it is fixed by the plantuml team. With the next plantuml release it should work like before

@startuml
!include https://raw.githubusercontent.com/plantuml-stdlib/C4-PlantUML/master/C4_Container.puml
!include <office/Servers/application_server>

AddContainerTag("webAppShadow", $bgColor="red", $sprite="application_server", $legendText="with shadow", $shadowing="true")
AddContainerTag("webAppNoShadow", $bgColor="green", $sprite="application_server", $legendText="no shadow", $shadowing="false")
AddContainerTag("webApp", $sprite="application_server", $legendText="default is no shadow")

Container(web_app_with, "Web Application with shadow", "Java, Spring MVC, Tomcat 7.x", $tags="webAppShadow")
Container(web_app_no, "Web Application no shadow", "Java, Spring MVC, Tomcat 7.x", $tags="webAppNoShadow")
Container(web_app_default, "Web Application default (is no shadow)", "Java, Spring MVC, Tomcat 7.x", $tags="webApp")

SHOW_LEGEND()
@enduml

Best regards Helmut

PS.: if you test it via https://www.plantuml.com/ and it is not working, then a cached response is used. In this case please add e.g. a space in the puml code and it should work