Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inconsistent spacing between attributes and methods in class diagram #5562

Closed
aubertc opened this issue Jun 6, 2024 · 1 comment · Fixed by #5880
Closed

Inconsistent spacing between attributes and methods in class diagram #5562

aubertc opened this issue Jun 6, 2024 · 1 comment · Fixed by #5880
Assignees
Labels
Graph: Class Status: Awaiting PR Type: Bug / Error Something isn't working or is incorrect

Comments

@aubertc
Copy link

aubertc commented Jun 6, 2024

Description

In the following:

classDiagram
class ClassName {
-attribute : type
- attribute : type

+GetAttribute() type
+ GetAttribute() type
}

the space is preserved between - and attribute, but not between + and GetAttribute:

image

Steps to reproduce

Simply enter the above code in an editor, like https://mermaid.live/edit#pako:eNptzzsOwjAMBuCrWJ5ALRfIhkBiY2HN4jZuGymPKnGGqurdCVAmWCz_nyzZXrGPhlFh7yjnq6UxkdfhneDyqnfyDKsOJxJJtivCoECWmSvBr-nQ3FjOXz8cd2_gL2_YoufkyZp6RF0DoFEm9qxR1dbwQMWJxs8oFYmPJfSoJBVuMcUyTqgGcrmmMhsS3p_YdXsCu55P9A

Screenshots

No response

Code Sample

No response

Setup

  • Mermaid version:10.9.1

Suggested Solutions

No response

Additional Context

No response

@aubertc aubertc added Status: Triage Needs to be verified, categorized, etc Type: Bug / Error Something isn't working or is incorrect labels Jun 6, 2024
@jgreywolf jgreywolf added Status: Approved Is ready to be worked on Graph: Class and removed Status: Triage Needs to be verified, categorized, etc labels Aug 27, 2024
@jgreywolf jgreywolf added Status: Awaiting PR and removed Status: Approved Is ready to be worked on labels Sep 25, 2024
@jgreywolf jgreywolf linked a pull request Sep 25, 2024 that will close this issue
4 tasks
@aubertc
Copy link
Author

aubertc commented Nov 1, 2024

Awesome, thanks a lot @jgreywolf @yari-dewalt @knsv ! Looking forward to seeing this feature makes it to the CLI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Graph: Class Status: Awaiting PR Type: Bug / Error Something isn't working or is incorrect
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants