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.

Sign up for GitHub

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

Jump to bottom

A link containing # in a header #3

Closed
rdrey opened this issue Sep 13, 2011 · 5 comments
Closed

A link containing # in a header #3

rdrey opened this issue Sep 13, 2011 · 5 comments

Comments

@rdrey
Copy link

rdrey commented Sep 13, 2011

I'm not sure whether this is a bug in marked or a symbol I need to escape, but parsing:

### Something [blabla](#/bla) is *great*.

results in a new paragraph starting and the header ending on the # in #/bla and doesn't give me a link (to #/bla) in the header.

Thanks for marked, I'm using it in coffeekup in a zappa project and it's awesome to parse markdown so easily.
@chjj
Copy link
Member

chjj commented Sep 13, 2011

Ah, yeah, that is definitely a bug. I'm on it.

@rdrey
Copy link
Author

rdrey commented Sep 13, 2011

sweet, thanks!

@chjj
Copy link
Member

chjj commented Sep 14, 2011

fixed 2c21b61

@chjj chjj closed this as completed Sep 14, 2011
@chjj
Copy link
Member

chjj commented Sep 14, 2011

also, this should now match the exact behavior of the original markdown. it was something that wasn't included in the test suite.

@rdrey
Copy link
Author

rdrey commented Sep 19, 2011

awesome! now i can generate clean html with fewer hacks.

tommie added a commit to tommie/marked that referenced this issue Sep 25, 2014
@tommie
Support HTML start tags with new lines in them.
ba059b2 
HTML5 Start tags can contain "one or more space characters" (8.1.2.1 markedjs#3).

A "space character" is space, tab, LF, FF or CR (2.4.1).

This change makes marked support any Unicode white space in tags by restricting
the autolink to not contain any /\s/ character. It is slightly more permissive
than the HTML5 standard, but that is unlikely to be a problem in practice.
tommie added a commit to tommie/marked that referenced this issue Sep 25, 2014
@tommie
Support HTML start tags with new lines in them.
b238897 
HTML5 Start tags can contain "one or more space characters" (8.1.2.1 markedjs#3).

A "space character" is space, tab, LF, FF or CR (2.4.1).

This change makes marked support any Unicode white space in tags by restricting
the autolink to not contain any /\s/ character. It is slightly more permissive
than the HTML5 standard, but that is unlikely to be a problem in practice.
@tommie tommie mentioned this issue Sep 25, 2014
Closed
gibson042 pushed a commit to gibson042/marked that referenced this issue Oct 20, 2017
@joshbruce
Merge pull request markedjs#3 from gibson042/document-xhtml
a34daa6 
Document the xhtml option
tommie added a commit to tommie/marked that referenced this issue Jan 22, 2018
@tommie
Support HTML start tags with new lines in them.
b093b5f 
HTML5 Start tags can contain "one or more space characters" (8.1.2.1 markedjs#3).

A "space character" is space, tab, LF, FF or CR (2.4.1).

This change makes marked support any Unicode white space in tags by restricting
the autolink to not contain any /\s/ character. It is slightly more permissive
than the HTML5 standard, but that is unlikely to be a problem in practice.
@UziTech UziTech mentioned this issue Mar 31, 2020
Merged
9 tasks
@calculuschild calculuschild mentioned this issue Jul 3, 2020
Merged
6 tasks
vagra referenced this issue in vagra/marked Oct 31, 2020
@vagra
can use both heading(#, ##, ###...) and kepan heading(#1, #2, #3...) …
9e1a7e4 
…at one time.
vagra referenced this issue in vagra/marked Nov 1, 2020
@vagra
update to upstream v1.2.2, can support both #, ##, ###... and #1, #2, #3
e9b9366 
... #108.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rdrey @chjj