search

Mermaid-Chart / issue-tracker

Other
7 stars 1 forks source link

Unable to rename a diagram #19

Closed oryx-it closed 8 months ago

oryx-it commented 9 months ago

I'm unable to change the name of a diagram.

I'm using a macOS 15.3.2

https://github.com/Mermaid-Chart/issue-tracker/assets/111514654/d65811be-efda-4939-bb16-29dd83490d9e

jipock-morgan commented 9 months ago

This is also happening to me - running Windows 11 / Microsoft Edge (116.0.1938.9).

DavidCampbell-architect commented 9 months ago

This happens to me as well. A workaround is to rename from the settings menu of the diagram. I'm running Windows 11, Brave Version 1.58.135 Chromium: 117.0.5938.140

nirname commented 9 months ago

Confirmed. The title of the diagram is not saved. After editing and going back to the diagrams list it is visible, but when you go back to editing you get "Untitled diagram" again image (13) image (14)

pbrolin47 commented 9 months ago

We did a fix related to diagram naming last week. Are you still having problems with this issue? Best regards, Per

nirname commented 9 months ago

It works.

But then again there is undesirable "Untitled diagram" text in the diagram name input. Plus when switching keyboard layout input loses focus. Perhaps these are 2 separate issues (apart from that one)

oryx-it commented 9 months ago

I don't how you're testing but it's still not working. The worst performing and most buggy product is the paid one :-(

https://github.com/Mermaid-Chart/issue-tracker/assets/111514654/f067f9cd-df51-4c14-8cfb-357860466a1b

DavidCampbell-architect commented 9 months ago

I still have the same problem as well. The only way to change the name is from the settings window for the diagram. Changing the name does not work if you change from within the diagram page.

pbrolin47 commented 8 months ago

Tested renaming of diagrams from within the digram page today. Unfortunately, I could not reproduce the problem. screen-capture.webm

nirname commented 8 months ago

Looks as if it is resolved, cannot reproduce either. Do you guys use integration tests? This case is a good candidate for that

pbrolin47 commented 8 months ago

Closing. Will reopen if needed