joplin icon indicating copy to clipboard operation
joplin copied to clipboard

Mermaid Pie Chart gets cut off

Open Daenges opened this issue 2 years ago • 1 comments

mermaid-1660924804126

The legend of Mermaid pie charts is truncated. This also applies to the editor, HTML and PNG export. Zoom level change does not fix this for me.

Environment

Joplin version: 2.8.8 Flatpak (Flathub) Platform: Linux OS specifics: Manjaro

Steps to reproduce

Here is a sample Graph, used to produce the image.

pie showData 
	title Anlageportfolio
	"Anleihen": 5000
	"Aktien": 5000

Describe what you expected to happen

Display a full graph.

Logfile

Console.log

Daenges avatar Aug 19 '22 16:08 Daenges

I could reproduce this. The Live Editor shows it correctly.

tessus avatar Sep 14 '22 12:09 tessus

mermaid-1664209358048

Same applies to the labeling of flowchart arrows.

Daenges avatar Sep 26 '22 16:09 Daenges

Hey there, it looks like there has been no activity on this issue recently. Has the issue been fixed, or does it still require the community's attention? If you require support or are requesting an enhancement or feature then please create a topic on the Joplin forum. This issue may be closed if no further activity occurs. You may comment on the issue and I will leave it open. Thank you for your contributions.

github-actions[bot] avatar Oct 27 '22 16:10 github-actions[bot]

This issue still persists.

Daenges avatar Oct 29 '22 11:10 Daenges

I could reproduce this. The Live Editor shows it correctly.

Here is another test on live editor renderer opened in full window

  1. open live editor rendered window

  2. resize the window to a lower the width (to 500px-600px )

  3. refresh the page

re-rendered graph gets cut off the same way as it gets cut in our app, looks like a problem from mermaid's side; viewbox width is being set lower than requiered in this case.

adarsh-sgh avatar Feb 11 '23 06:02 adarsh-sgh

Maybe you can open a ticket in the mermaid repo?

tessus avatar Feb 11 '23 13:02 tessus

Maybe you can open a ticket in the mermaid repo?

Sure, will do it.

edit: https://github.com/mermaid-js/mermaid/issues/4082

adarsh-sgh avatar Feb 11 '23 14:02 adarsh-sgh

Fixed by #8890

tessus avatar Sep 19 '23 13:09 tessus