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

Two small tweaks I found essential #1

Merged
merged 1 commit into from
Jan 9, 2013
Merged

Conversation

acuthbert
Copy link
Contributor

Fixed issue with two back ticks triggering curly quotes if above a code block but triggering inline code tags if below a code block

Changed path for GraphViz on windows to use Program Files (x86) as a more modern default

…de block but triggering inline code tags if below a code block!

Changed path for GraphViz on windows to use Program Files (x86) as a more modern default
@edwinm
Copy link
Owner

edwinm commented Dec 29, 2012

Your change "$result .= SmartyPants(Markdown($md));" is already in the latest version.

Your other change with a different path (Program Files (x86)): it would be better if it was applied dynamically, at runtime, so users with and without the "(86)" part can use the code.

@acuthbert
Copy link
Contributor Author

Hi edwinm, the call is made twice - the current version has it correct in one of these (around line 110 I think) but the other earlier call still is the wrong way round.

edwinm added a commit that referenced this pull request Jan 9, 2013
Two small tweaks I found essential
@edwinm edwinm merged commit 59ab705 into edwinm:master Jan 9, 2013
@edwinm
Copy link
Owner

edwinm commented Jan 9, 2013

You're right. Thanks for the fix.

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