<cite> and <code> are allowed by the default filter, and drupal_html_to_text() should support them.
This is a bug report, not a feature request, IMO.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | mail.inc3_.patch | 1.54 KB | vladimir.dolgopolov |
| #3 | mail.inc2_.patch | 2.5 KB | vladimir.dolgopolov |
| #1 | mail.inc_.patch | 805 bytes | vladimir.dolgopolov |
Comments
Comment #1
vladimir.dolgopolov commentedHope the patch helps.
Comment #2
salvisNo, this is too simple. Maybe it works for <code>, which should just be rendered as it's entered, with all spaces and newlines intact (but I suspect that it's line-wrapped), but <cite> needs to be emphasized in some way, at least with double quotes.
Comment #3
vladimir.dolgopolov commentedThe patch renders:
code - 4 spaces indent, surrounded by a newline, preserve all spaces
cite - surround by a 'sharp' symbol, e.g.
What we can do with a long lines in 'code' ?
I suggest add the '/' symbol at the end of each wrapped line. But maybe it is not good.
Comment #4
Steven commentedCode tags do not preserve white space in HTML. They should be treated like any other inline tag.
Comment #5
vladimir.dolgopolov commentedOh, it's true.
Well, how to emphasize inline code tags here?
1) You can use the %code lnline% tag to define computer code.
2) You can use the {code lnline} tag to define computer code.
3) You can use the [code lnline] tag to define computer code.
4) You can use the -code lnline- tag to define computer code.
What tag should be used for multiline code snippets, 'pre' ?
Comment #6
Steven commentedThere is no HTML tag that does what people expect. Pre only preserves white space, but still requires ampersands and angles to be escaped. This is why codefilter exists.
Comment #7
vladimir.dolgopolov commentedI've consulted with http://en.wikipedia.org/wiki/Lightweight_markup_language and have decided:
The patch provides this.
Comment #8
dwwBy default, every browser I've ever known renders
<cite>in italics, so it seems like for consistency, we should use /cite/ just like we do for<i|em>. Also, note that you need to support attributes in these tags. See #258192: drupal_html_to_text() doesn't support <strong class="foo">Comment #9
dpearcefl commentedDoes this issue exist in current D6?
Comment #10
dpearcefl commented