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
72.36k stars 6.59k forks source link

Vertical line is not hidden for note over actor in sequence diagram #5736

Open goplayerjuggler opened 2 months ago

goplayerjuggler commented 2 months ago

Description

The vertical line for a process is hidden by a note over that process; but it isn't hidden for a note over an actor. It should be; it looks ugly and gets in the way of reading the diagram.

Steps to reproduce

The following code shows the problem.


sequenceDiagram 
    actor A 
    note over A: the vertical line isn't hidden by this note! <br/>:(
    A->>B:message
    note over B: No vertical line: it's fine <br/>:)

Screenshots

image

Code Sample

No response

Setup

Suggested Solutions

No response

Additional Context

I reported this yesterday by mistake to a different repo: https://github.com/tomoyukim/vscode-mermaid-editor/issues/106

KawtharAlakri commented 2 months ago

Hi @goplayerjuggler are you using any custom config? Because I can't re-produce your issue, it's rendering perfectly for me. image

goplayerjuggler commented 2 months ago

No but I'm behind a corporate VPN, maybe that could be related to this issue?

Here's the code, Austin-Fulbright asked for it (maybe? i got an email alert, but I don't see AF's request here...)

sequenceDiagram 
    actor A 
    note over A: the vertical line isn't hidden by this note! <br/>:(
    A->>B:message
    note over B: No vertical line: it's fine <br/>:)
goplayerjuggler commented 2 months ago

@KawtharAlakri no custom config either.

KawtharAlakri commented 2 months ago

which theme are you using ?

goplayerjuggler commented 2 months ago

The default one - the Light theme.

Update: this morning I can no longer repro the bug!