Please replace with the current version of the sandbox. See the difference here. Specifically, I want to replace

}} }}{{#ifexpr:{{#ifeq:{{NAMESPACE}}|User talk|1|0}}*{{#ifeq:{{{disclaimer|}}}|yes|1|0}}| {{usertalkpage}} }}

with

}}}} {{#ifeq:{{{disclaimer|}}}|yes|{{#ifeq:{{NAMESPACENUMBER}}|1|{{usertalkpage}}}}}}

to simplify the expression by chaining the ifs rather than multiplying the results and by testing the disclaimer parameter first, since it is less likely to be called. Note that, while here it appears as one line, there are multiple lines of parser function syntax that I am requesting be changed (see in edit mode); I couldn't figure out how to get it to separate the lines. Thanks, --DannyS712 (talk) 00:17, 16 March 2019 (UTC)[reply]

 Done — Martin (MSGJ · talk) 16:49, 17 March 2019 (UTC)[reply]

MSGJ, I'm not exactly sure how/why, but I think your most recent edit may have added a blank line to the template, which can now be seen at the top of talk pages. ---Another Believer _(Talk) 17:30, 17 March 2019 (UTC)[reply]

This edit should fix it. This is why we prefer proposed changes to templates to be made in the /sandbox subpage first and demonstrated in the /testcases subpage. Code blobs in a talk page cannot provide such testing. --Redrose64 🌹 (talk) 21:51, 17 March 2019 (UTC)[reply]

@Redrose64: Sorry --DannyS712 (talk) 21:52, 17 March 2019 (UTC)[reply]

@Redrose64: Speaking of not testing, and I'm really sorry, but it should have been namespace 3, not 1 ({{#ifeq:{{NAMESPACENUMBER}}|3|{{usertalkpage}}}}}}) I'm really sorry for not trying it in the sandbox first --DannyS712 (talk) 07:31, 19 March 2019 (UTC)[reply]

 Done --Redrose64 🌹 (talk) 19:37, 19 March 2019 (UTC)[reply]

Add Category:Templates that are not mobile friendly. This template isn't working on mobile. -- CptViraj (📧) 17:39, 4 August 2019 (UTC)[reply]

 Done DannyS712 (talk) 18:57, 4 August 2019 (UTC)[reply]

The template definition contains a <table> element with four rows. The first and last rows contain a single cell element <td> each, which declare spanning across four columns: colspan=4. However, the remaining two rows contain only three cells, or even just two (depending on the template's arpol parameter).

That causes the W3C HTML validator to issue a warning for the row number 2:

A table row was 2 columns wide, which is less than the column count established by the first row (4).

and an error for the whole table:

Table columns in range 3…4 established by element td have no cells beginning in them.

See https://validator.w3.org/nu/?doc=https%3A%2F%2Fen.wikipedia.org%2Fwiki%2FTemplate%3ATalk_header

Possible solution: reduce the colspan setting to 2, conditionaly to 3, depending on arpol. --CiaPan (talk) 11:15, 19 January 2020 (UTC)[reply]

P.S.
The validator also complains about a duplicated ID talkheader, but that results from an indirect code duplication (the template is displayed at the template's page directly, and then included by an included /doc subpage) and doesn't need fixing. --CiaPan (talk)

@CiaPan: Normally there are two columns. But under certain circumstances, one or two extra columns are added: if shortcuts are specified, or when |arpol=yes is set. --Redrose64 🌹 (talk) 11:55, 19 January 2020 (UTC)[reply]

@Redrose64: Yes, I suppose that's exactly what I described above. Any suggestions about improving it? I'd like to fix it myself, but I can't due to the template protection.... --CiaPan (talk) 15:42, 19 January 2020 (UTC)[reply]

@Redrose64: Oh, now I see I overlooked the case of four cells. But that's not a problem - the error isn't in just declaring a span of four columns, but rather in declaring four columns when there are actually just two or three. --CiaPan (talk) 17:06, 19 January 2020 (UTC)[reply]

Indeed, you can't edit the live template - but you can edit its sandbox, which I have synchronised. Just follow the directions at WP:TESTCASES. --Redrose64 🌹 (talk) 14:12, 20 January 2020 (UTC)[reply]

@Redrose64: Thank you! Changes done: Special:Diff/936846961. Could you review, please? --CiaPan (talk) 10:56, 21 January 2020 (UTC)[reply]

The gap at the top is significantly larger. --Redrose64 🌹 (talk) 21:49, 21 January 2020 (UTC)[reply]

@Redrose64: After two hours or so of googling and experimenting under F12 with style attributes for the whole cascade (td - table - tr - td) I found it's enough to set style="border-spacing:0" for the inner table. However, when I tried to save the change, an edit conflict popped out, and I gave up. Feel free to close this thread as unresolved, rejected or whatever you like. --CiaPan (talk) 22:36, 21 January 2020 (UTC)[reply]

Figured out why CiaPan's version added an extra space. When that version was expanded, the last four lines were

</tr></table></td></tr>


</table>

For some reason, MediaWiki interprets these empty lines as misplaced wiki markup and puts it outside the table. When the live template is expanded, the last four lines are

</tr>


</table>

and these empty lines are, again for some reason, not interpreted as markup. So simply inserting a line break before </tr> got rid of the gap. Nardog (talk) 23:15, 21 January 2020 (UTC)[reply]

Using multiple tables is counterproductive. Rather, the number of colspan should be responsive to shortcuts and |arpol=. Nardog (talk) 21:51, 21 January 2020 (UTC)[reply]

I believe this does the job, though it kinda feels like bringing a cannon to a knife fight. Nardog (talk) 22:14, 21 January 2020 (UTC)[reply]

Take a look at Template:Talk header/testcases. The shortcut box is not aligning the to right. Also, the right margin in the white cell is too small (the relative widths of the white and tan cells are also different than in the original, but I'm less concerned about that). --Bsherr (talk) 00:51, 22 January 2020 (UTC)[reply]

The latest revision of the sandbox at the moment is my correction to CiaPan's version, not mine. Nardog (talk) 02:57, 22 January 2020 (UTC)[reply]

On User talk:Evad37/rater.js, the talk page header says that "you can send messages and comments to Evad37/rater.js." Is there a way that can make the user name display on the banner (i.e. Evad37) instead of the full name (i.e. Evad37/rater.js)? Thanks, from TLOM (The Lord of Math) (Message) 04:23, 31 March 2020 (UTC)[reply]

We would need to amend this:

This is [[{{SUBJECTPAGENAME}}|{{PAGENAME}}]]'s [[Wikipedia:Talk page guidelines|talk page]], where you can send messages and comments to {{PAGENAME}}.

to strip out the subpages. I need to go to work soon. --Redrose64 🌹 (talk) 08:12, 31 March 2020 (UTC)[reply]

@Redrose64 and The Lord of Math: I've fixed the issue in the sandbox here. Please adopt the sandbox version if it looks good. Cheers,