Hair space or thin space? edit

FYI, I've started a discussion on the Manual of Style for whether this template (and others) should put a hair space or a thin space after the attribution dash. (Apologies if I'm just totally misunderstanding something.) — ⁠Will ⁠• ⁠B[talk] 20:57, 7 May 2023 (UTC)Reply

Template-protected edit request on 3 July 2023 edit

Change the thin space ( ) to a hair space ( ), per WP:Manual of Style § Other uses (em dash only), which recommends attribution em dashes be followed by a hair space in block quotations. {{Cquote}} and {{Quote frame}} both use the hair space; this template is the only one I could find that uses a thin space. — Will • B[talk] 01:37, 4 July 2023 (UTC)Reply

So turns out that in 2015, SMcCandlish replaced the NBSPs in several quote templates (including this one) with thin spaces—not hair spaces, which "apparently [did] not render in all browsers". But I guess that's since improved, because in 2019, he replaced them with hair spaces on {{Cquote}} and {{Quote frame}}. But the rest of the quotation templates still had the thin space, so I've replaced them all with hair spaces except on the two protected templates—this one, and also {{Talk quote block}}. (And yes, I've put way too much effort into this change that nobody will ever notice. But that's kind of my thing.) — Will • B[talk] 03:23, 4 July 2023 (UTC)Reply
Yeah, the hair-space rendering problem is very old news at this point; we should use hair-space for this consistently.  — SMcCandlish ¢ 😼  03:33, 4 July 2023 (UTC)Reply
  Done. I fixed it in both of the mentioned templates.  — SMcCandlish ¢ 😼  03:39, 4 July 2023 (UTC)Reply
Awesome thanks. It looks like there's one more spot on {{Talk quote block}} that still uses a standard non-breaking space after the em dash—the one that goes with the |source= parameter, I think. Could you fix that too? — Will • B[talk] 03:56, 4 July 2023 (UTC)Reply

Template-protected edit request on 13 October 2023 edit

{{#if:{{{class|}}}{{{id|}}}{{{diff|}}}{{{4|}}}{{{5|}}}|{{main other|[[Category:Pages incorrectly using the Blockquote template]]}}
+
{{#if:{{{class|}}}{{{id|}}}{{{diff|}}}{{{5|}}}|{{main other|[[Category:Pages incorrectly using the Blockquote template]]}}

I see no reason for the usage of the positional alias for source to be marked as incorrect. Aaron Liu (talk) 13:44, 13 October 2023 (UTC)Reply

As far as I can tell, the "incorrectly" tracking category was installed during a template merge. Now that the category is cleared, it is no longer needed. All of the numbered parameters, as fragile as they are, are supported. I have removed the "incorrectly" tracking entirely and removed |id= and |diff=, two parameters kept temporarily during a template merge, from the unknown parameter check. – Jonesey95 (talk) 15:24, 13 October 2023 (UTC)Reply

Blockquote tag vs this template: strange indentation on mobile edit

Recently the <blockquote> tag has been producing some additional indentation on mobile:

Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.

This template doesn't do that, even though AFAIK it uses the blockquote element:

Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.

What happened? Can this be fixed? Hairy Dude (talk) 12:34, 24 February 2024 (UTC)Reply

In mobile, the blockquote tag appears to include style="margin-left:41px. In the rendered template, that style is not included for some reason. I do not know how to get our CSS file to apply a style only in mobile view. – Jonesey95 (talk) 16:08, 24 February 2024 (UTC)Reply