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

IE, local html, cssRules access is denied #155

Merged
merged 1 commit into from
Apr 19, 2015
Merged

IE, local html, cssRules access is denied #155

merged 1 commit into from
Apr 19, 2015

Conversation

tylerlong
Copy link
Collaborator

IE, open html file directly from hard disk(insead of from a http server), cssRules access is denied.

Thus an expection is thrown. But in the catch statement, another one is thrown again thus cause the whole mermaid code break.

In this line console.warn('Invalid CSS selector "' + rule.selectorText + '"', err); rule is undefined, so exception will be thrown if we don't check.

IE, open html file directly from hard disk(insead of from a http server), cssRules access is denied.

Thus an expection is thrown. But in the catch statement, another one is thrown again thus caused the whole mermaid code break. 

In this line `console.warn('Invalid CSS selector "' + rule.selectorText + '"', err);`  rule is undefined, so exception will be thrown if we don't check.
@knsv
Copy link
Collaborator

knsv commented Apr 19, 2015

Thanks! Good catch!

knsv added a commit that referenced this pull request Apr 19, 2015
IE, local html, cssRules access is denied
@knsv knsv merged commit 11a30dd into mermaid-js:master Apr 19, 2015
mgenereu pushed a commit to mgenereu/mermaid that referenced this pull request Jun 25, 2022
…yarn/url-parse-1.5.1

Bump url-parse from 1.4.7 to 1.5.1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants