Skip to content
This repository has been archived by the owner on Nov 27, 2019. It is now read-only.

fix(docs): removed parenthesis from fullpage links #28

Merged
merged 2 commits into from
Oct 30, 2019

Conversation

evwilkin
Copy link
Member

Fixes #18

@@ -53,7 +53,7 @@ const Example = props => {
const html = Prism.highlight(children, Prism.languages.javascript, 'javascript');
return <pre dangerouslySetInnerHTML={{ __html: html }} />;
}
const fullscreenLink = `${location.pathname}/${title.toLowerCase()}`;
const fullscreenLink = `${location.pathname}/${title.toLowerCase().replace(/[()]/g,'')}`;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use helpers/slugger.js

@redallen redallen merged commit 8cab36e into patternfly:master Oct 30, 2019
@redallen
Copy link
Contributor

🎉 This PR is included in version 1.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

support example titles with parentheses
3 participants