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

fix escaping of title when generating a markdown link #2456

Merged
merged 1 commit into from Feb 7, 2020

Conversation

tessus
Copy link
Collaborator

@tessus tessus commented Feb 7, 2020

Previously a title with brackets was escaped incorrectly. The brackets were replaced by underscores.

The following title title [square] (round) looked like this:

[title _square_ _round_](:/c54794f53e5e4b1aa558699e255d5f95)

Now it looks like this:

[title \[square\] (round)](:/c54794f53e5e4b1aa558699e255d5f95)

image

fixes #2085

Previously a title with brackets was escaped incorrectly. The brackets were replaced by underscores.

The following title `title [square] (round)` looked like this:

[title _square_ _round_](:/c54794f53e5e4b1aa558699e255d5f95)

Now it looks like this:

[title \[square\] (round)](:/c54794f53e5e4b1aa558699e255d5f95)

fixes laurent22#2085
@tessus tessus added bug It's a bug PR-tested labels Feb 7, 2020
@tessus
Copy link
Collaborator Author

tessus commented Feb 7, 2020

@laurent22 Oh, I just noticed that I used my name for the tests. I wrote the tests before the code, so I just used the first thing that came to mind. I can change it to the example I used when I wrote the commit message and description.

@laurent22
Copy link
Owner

As long as the tests are clear, and they are, I'm fine with it, so let's merge. Thanks for the fix!

@laurent22 laurent22 merged commit 3f23d8e into laurent22:master Feb 7, 2020
@tessus tessus deleted the fix/esacpe-title branch February 8, 2020 00:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug It's a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Parenthesis in title change to underscores when using "Copy Markdown Link"
2 participants