dcwatson / bbcode

A pure python bbcode parser and formatter.
BSD 2-Clause "Simplified" License
68 stars 17 forks source link

Some tag values should not be stripped by default #7

Closed marcinn closed 10 years ago

marcinn commented 11 years ago

Let's take source text: Normal font [b]and bold font [/b]and again normal

As the result "bold font" will be joined with "and again" without space. But expected result is <strong>and bold font </strong>and, because many non-dev users have no idea about stripping tag values (they even don't now anything about tags and bbcode).

Version: 1.0.10

dcwatson commented 11 years ago

I think you're probably right. I changed TagOptions.strip to default to False in 80ab44c, but kept lists and quotes set to strip, since they're typically block-level elements anyway. I think strip=False is probably a safer (at least more predictable) default for new tags, anyway. If you want to pull down the latest and try it out, I can release a new version to PyPI in the next couple days.