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

Live Editor: Permalink address not being parsed #202

Closed
ichabodcole opened this issue Sep 3, 2015 · 4 comments
Closed

Live Editor: Permalink address not being parsed #202

ichabodcole opened this issue Sep 3, 2015 · 4 comments
Labels
Contributor needed Type: Bug / Error Something isn't working or is incorrect

Comments

@ichabodcole
Copy link

I'm expecting that clicking the permalink button in the live editor should allow me to then copy the url, paste in a new tab and see the graph I created, but I'm only seeing the default graph when doing this.

  1. Go to http://knsv.github.io/mermaid/live_editor
  2. Make a small change to the default diagram.
  3. Click the Permalink button, copy the url, and paste into a new tab.
  4. Expecting diagram to show the changes, but the default is still displayed.

Tested in chrome and safari on OSX

Seeing this errror in the console as well.

Invalid CSS selector "[ng:cloak], [ng-cloak], [data-ng-cloak], [x-ng-cloak], .ng-cloak, .x-ng-cloak, .ng-hide:not(.ng-hide-animate)" DOMException: Failed to execute 'querySelectorAll' on 'Element': '[ng:cloak], [ng-cloak], [data-ng-cloak], [x-ng-cloak], .ng-cloak, .x-ng-cloak, .ng-hide:not(.ng-hide-animate)' is not a valid selector.(…)
@knsv
Copy link
Collaborator

knsv commented Sep 5, 2015

Hmm. This used to work.

@naseer is this something you could help us with? If so, much appreciated.

@knsv knsv added Type: Bug / Error Something isn't working or is incorrect Contributor needed labels Sep 15, 2015
@naseer
Copy link

naseer commented Oct 14, 2015

just tried it out - it seems to work fine, I see the link text getting updated. I tried on Chrome and Safari on OSX

@spect88
Copy link
Collaborator

spect88 commented Oct 24, 2015

@ichabodcole Is this still a problem? With the new permalinks, there's a high chance this problem is fixed.

@orschiro
Copy link

@spect88

Just tested the latest version. It is working great now.

Thank you!

~ Robert

@spect88 spect88 closed this as completed Oct 27, 2015
mgenereu pushed a commit to mgenereu/mermaid that referenced this issue Jun 25, 2022
…yarn/develop/cypress-7.5.0

Bump cypress from 7.4.0 to 7.5.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Contributor needed Type: Bug / Error Something isn't working or is incorrect
Projects
None yet
Development

No branches or pull requests

5 participants