Page MenuHomePhabricator

Placing table syntax outside the main body of a Page: (on Wikisource) causes a stray <div> tag to be generated
Closed, DuplicatePublic


is apparently in error being reported as having a "Missing end tag" for a DIV.

Having expanded out the code in use, there are no mismatched tag to be located in any of the contributor supplied markup.

This suggests that the placing of the opening of a table in the header is incorrectly placing it inside the header DIV (i.e the header portion marked by a special instance of <noinclude></noinclude>. which is generating badly formed HTML on rendering. However this approach has been used on a number of other pages without an issue being identifed (as yet) so I'd like a clear explanation of why the SPECIFC markup here is creating issues.

I am seeing the same problem with

Again there is no mismatch of tags in the contributor supplied markup according to the advice given in the project concerned.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJan 18 2018, 3:40 PM
Ankry added a subscriber: Ankry.Jan 18 2018, 3:59 PM
<subbu> so, on that Page:Cowie's page ... the problem is the {{nop}} in the first line ... it generates a <div></div> .. but it is embedded inside a table outside a table-cell .. i.e. there is no preceding | 
<subbu> so, it gets "fostered" out of the table and also causes the missing </div> error .. but, the core problem is that the {{nop}} there is out of place. it doesn't belong there.
<ShakespeareFan00> subbu: Yet that {{nop}} is the advised approach for split tables
<ShakespeareFan00> According to the disscussion at Wikisource
<ShakespeareFan00> If you'd like to tell the long-standing and respected contributor there that THEY are wrong
<ShakespeareFan00> I wish you the best of luck
<subbu> ShakespeareFan00, maybe so .. but it has to be proper markup. if it is broken markup, nothing the parser / browser can do with it. i cannot wade into that discussion now, but maybe the instructions are missing steps / are being misinterpreted.
<ShakespeareFan00> subbu:  Without the {{nop}} there's no line-feed so that the starting |-
<ShakespeareFan00> gets rendered into the last cell of the preceeding table row
<ShakespeareFan00> due to the way Proofread Page works
<ShakespeareFan00> This mangles all kinds of tables
<ShakespeareFan00> The correct way of doing this would be to have "proper" directives to use changing the parser handling
<ShakespeareFan00> but when I suggested the same a while back there absolutely no interest
<ShakespeareFan00> (sigh)
ssastry triaged this task as Normal priority.Jan 18 2018, 4:12 PM
Aklapper renamed this task from Placing table synatax outside the main body of a Page: (on Wikisource) causes a stray DIV tag to be generated. to Placing table syntax outside the main body of a Page: (on Wikisource) causes a stray <div> tag to be generated.Jan 18 2018, 7:03 PM