search

mermaid-js / mermaid

Generation of diagrams like flowcharts or sequence diagrams from text in a similar manner as markdown
https://mermaid.js.org
MIT License
70.81k stars 6.35k forks source link

Remove redundant surrounding classbox from lollipop #4037

Open mestaritonttu opened 1 year ago

mestaritonttu commented 1 year ago

Description

This is a continuation of #1145. As mentioned in #3311 there is still a classbox to be removed.

Steps to reproduce

Check the class diagram demos, syntax is Interface1 ()-- Interface1Impl

Screenshots

No response

Code Sample

No response

Setup

No response

Additional Context

No response

tomperr commented 1 year ago

I am a little confused as to what is expected, as I don't really understand what has been done and what is yet to be implemented. Can you please give some explanations? Sorry if I am missing something obvious

mestaritonttu commented 1 year ago

I am a little confused as to what is expected, as I don't really understand what has been done and what is yet to be implemented. Can you please give some explanations? Sorry if I am missing something obvious

See these examples of how it should look like:

https://www.uml-diagrams.org/realization.html

https://plantuml.com/class-diagram#8b64d7547fa4f699

At the head of the lollipop, there is only text, not a box. So in the Mermaid class diagram section in the demo page, Interface1, Animal and Fruit should not be inside a box.