Last modified: 2013-07-07 00:02:24 UTC
The following template in http://en.wikipedia.org/wiki/Jonestown,_Texas expands with a spurious indent pre: {{unreferenced section|date=May 2011}}
Pretty rare and can very likely be worked around by fixing the template. Lowering priority for now.
*** Bug 50580 has been marked as a duplicate of this bug. ***
This is caused by inconsistent (in my view) PHP parser behavior while handling indent-pres in tables. See example below: {| | This generates an indent-pre |} <table><tr> <td>This does not generate an indent-pre </td></tr></table> However, Parsoid currently introduces indent-pres in both cases. The template https://en.wikipedia.org/w/index.php?title=Template:Unreferenced_section&oldid=528729873&action=edit does not introduce an indent-pre because of this inconsistency. A reasonable fix in this case might be to remove the newline after |section in the above template.
A more reasonable default behavior might be not to generate indent-pres in table-cells at all, since there are at least 2 scenarios currently where leading whitespace in table-cells doesn't result in indent-pres. See below. 1. Indent-pre: {| |- | foo |} 2. No indent-pre: {| |- foo |} 3. No indent-pre: <table><tr><td> foo </td></tr></table> However, not sure if there are pages out there that exploit behavior 1.
James fixed the template. I verified that it is fixed locally. Now, we just wait for cached html to be purged for all dependent pages. At some point, worth considering the <pre> in td-cells issue and see if we should change that behavior. For now, closing.
*** Bug 50640 has been marked as a duplicate of this bug. ***
At this time, the last remaining issue is that cached HTML didn't get purged when the template was updated. If we are in a rush, we could simply get Roan to purge the caches and it will get fixed. The page in bug 50640 works for me (for some reason different from the screenshot that Oliver sees).
*** Bug 50069 has been marked as a duplicate of this bug. ***
