[ 
https://issues.apache.org/jira/browse/FLINK-2127?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14576231#comment-14576231
 ] 

Maximilian Michels commented on FLINK-2127:
-------------------------------------------

It seems like the real issue is not the {{br}} tag but that the Kramdown parser 
does not parse html in markdown paragraphs (blocks). You need to explicitly 
start a new html block (i.e. by inserting a new line). In my opionion, this is 
fine because it keeps markdown and html separated from each other.

So in the example, just add a new line before the {{p}} tag. I fixed that on 
the master.

What bothers me is that the Kramdown parser's specification actually states the 
following: 

{quote}
The original Markdown syntax specifies that an HTML block must start at the 
left margin, i.e. no indentation is allowed. Also, the HTML block has to be 
surrounded by blank lines. Both restrictions are lifted for kramdown documents.
{quote}
http://kramdown.gettalong.org/syntax.html#html-blocks

So, although we are using Kramdown, the parsing behaves more like the original 
markdown specification, i.e. html code needs to be separated from other 
markdown text. I verified whether this is really Kramdown's behavior by 
executiong the following in a Ruby shell:

{code}
require 'kramdown'
Kramdown::Document.new("test 1 2 3 <p>bla bla bla</p>\n<p>bla bla 
bla</p>").to_html
=> "<p>test 1 2 3 &lt;p&gt;bla bla bla&lt;/p&gt;</p>\n<p>bla bla bla</p>\n"
{code}

Interestingly, it confirms that only html code which is separated by a new line 
gets parsed as actual html code. I'm stumped :)

> The GSA Documentation has trailing </p> s
> -----------------------------------------
>
>                 Key: FLINK-2127
>                 URL: https://issues.apache.org/jira/browse/FLINK-2127
>             Project: Flink
>          Issue Type: Bug
>          Components: Documentation, Gelly
>    Affects Versions: 0.9
>            Reporter: Andra Lungu
>            Assignee: Maximilian Michels
>            Priority: Minor
>             Fix For: 0.9
>
>
> Within the GSA Section of the documentation, there are trailing: <p 
> class="text-center"> image </p>. 
> It would be nice to remove them :) 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to