[
https://issues.apache.org/jira/browse/THRIFT-2375?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Nevo Hed updated THRIFT-2375:
-----------------------------
Attachment: out_master.html
out_090.html
> Excessive <br>'s in generated HTML
> ----------------------------------
>
> Key: THRIFT-2375
> URL: https://issues.apache.org/jira/browse/THRIFT-2375
> Project: Thrift
> Issue Type: Bug
> Components: HTML - Compiler
> Affects Versions: 0.9.1
> Reporter: Nevo Hed
> Priority: Minor
> Attachments: THRIFT-2375.thrift, out_090.html, out_master.html
>
>
> This was working "right" (at least how I thought it should work) back in
> 0.9.0 going back at least to 0.7.0 and introduced in 0.9.1
> https://git-wip-us.apache.org/repos/asf?p=thrift.git;a=commit;h=63e3c63
> The issue is that there is a '<br>' in HTML after every newline in the source
> comment (thrift). I assume that this was not the intention of the
> THRIFT-1800 change.
> I will attach complete sample thrift files and html output, but here is a
> summary:
> This is what the output USED to look like
> {quote}
> <tr><td>1</td><td>theThing</td><td><code>i32</code></td><td>Some comments can
> go for quite a while and may span multiple
> lines. What looks like a good spot to break a line in the
> thrift file may turn out to be not so great in HTML. In
> fact in HTML we should let the browser decide when to start
> a new lines. Users can still
> insert a break<br> when they really need/want it
> </td><td>required</td><td></td></tr>
> </table><br/></div></div></body></html>
> {quote}
> And this is what it looks like today
> {quote}
> <tr><td>1</td><td>theThing</td><td><code>i32</code></td><td>Some comments can
> go for quite a while and may span multiple<br/>lines. What looks like a good
> spot to break a li
> ne in the<br/>thrift file may turn out to be not so great in HTML.
> In<br/>fact in HTML we should let the browser decide when to start<br/>a new
> lines. Users can still<br/>
> insert a break<br> when they really need/want
> it<br/></td><td>required</td><td></td></tr>
> </table><br/></div></div></body></html>
> {quote}
--
This message was sent by Atlassian JIRA
(v6.1.5#6160)
