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

Architecture diagram does not support non-alphanumeric characters in titles #5928

Open
wizbit opened this issue Oct 4, 2024 · 0 comments · May be fixed by #5929
Open

Architecture diagram does not support non-alphanumeric characters in titles #5928

wizbit opened this issue Oct 4, 2024 · 0 comments · May be fixed by #5929
Labels
Status: Triage Needs to be verified, categorized, etc Type: Bug / Error Something isn't working or is incorrect

Comments

@wizbit
Copy link

wizbit commented Oct 4, 2024

Description

The following definition fails to render. This is due to the architecture diagram only supporting alphanumeric and spaces for titles.

architecture-beta 
  service api(internet)[https://mermaid.js.org/]

Steps to reproduce

  1. Put the following into the live editor.
      architecture-beta 
        service api(internet)[https://mermaid.js.org/]
    
  2. Rendering fails with Error: Parsing failed: unexpected character: ->[<- at offset: 42, skipped 1 characters.

Screenshots

No response

Code Sample

https://develop.git.mermaid.live/edit#pako:eNotjDEOAiEQRa9CptJkXXpqb2ApFiOMgApshsHEbPbuYrLd_y8vbwVXPYEBZBeTkJPOdLqToLJFqUb8SY4ULumQihAXkuM1iizNaJ2JMyY_P9tcOeibLTDBDkdy_RcsSKRMFsyYHvllwZZteNilXr7FgRHuNAHXHiKYB77beH3xKHROGBjzTrcfiyU8VA

Setup

  • Mermaid version: 11.3.0
  • Browser and Version: All

Suggested Solutions

packages/parser/src/language/architecture/architecture.langium#L53 should be replaced with terminal ARCH_TITLE: /\[[^\]]+\]/;

Additional Context

No response

@wizbit wizbit added Status: Triage Needs to be verified, categorized, etc Type: Bug / Error Something isn't working or is incorrect labels Oct 4, 2024
@wizbit wizbit linked a pull request Oct 4, 2024 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Triage Needs to be verified, categorized, etc Type: Bug / Error Something isn't working or is incorrect
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant