Module talk:Sidebar

Add support for V T E

Can you add support for V T E {{navbar|Template Name|mini=y}} in the bottom right corner of this template like many other templates have? Qwerty284651 (talk) 02:32, 5 June 2024 (UTC)

@Qwerty284651: It's called a navbar, and it's already present. Why do you think that it isn't? --Redrose64 🌹 (talk) 15:37, 5 June 2024 (UTC)

I meant to add to the {{sidebar}} template the ability display VTE i.e. the {{navbar|Template Name|mini=y}}. Qwerty284651 (talk) 16:28, 5 June 2024 (UTC)

Same answer, same question. It's already present in Template:Sidebar, so, why do you think that it isn't? If something isn't working as expected. you need to provide examples. --Redrose64 🌹 (talk) 18:16, 5 June 2024 (UTC)

I don't see VTE in {{Tennis tournaments}}. Qwerty284651 (talk) 18:22, 5 June 2024 (UTC)

You need a |name= parameter, like this. It is documented, at Template:Sidebar#Parameters, particularly in the paragraph before the table. --Redrose64 🌹 (talk) 20:19, 5 June 2024 (UTC)

Urgent: Please fix this template for printed content Module:Sidebar/styles.css.

Firstly, apologies for writing in English if this is not your first language (this is an automated message).

This template has been detected as one of 436 pages using styles that break the page when printed when the user is using dark mode. The fix is very straightforward - all your styles relating to dark mode must be scoped to. Since there is a high risk of this templates being copied to other wikis it is important this notice is acted on ASAP.

To fix this:

1. Update `@media (prefers-color-scheme: dark` to `@media screen and (prefers-color-scheme: dark`
2. Wrap any styles relating to `html.skin-theme-clientpref-night` in `@media screen`

If this message has not been acted on in 7 days, this will be fixed by an automated script. Thank you for your help fixing this important issue.

For any questions feel free to ask them at phab:T369874.

Jon (WMF) (talk) 18:22, 2 August 2024 (UTC) on behalf of the web team.

Missing CSS variable?

On row 385 there is the line :cssText('color: var(--color-base)'), but the --color-base CSS variable doesn't seem to be defined anywhere? MKFI (talk) 13:18, 15 October 2024 (UTC)

@Jdlrobson: I see that you added this. Where is the CSS variable defined? And why is this an inline style declaration instead of a class? Now dependent templates have problems changing text colors without using !important CSS declarations. MKFI (talk) 11:20, 17 October 2024 (UTC)

The variable is defined in the mediawiki design system. It is active on the default skins of Vector 2022 and Minerva and is essential to make this template work in dark mode. It is not defined on opt in skins such as Monobook and Vector and there it will be ignored (we can add a fallback if that is useful and less confusing).

this link should have lots of helpful context. Let me know if you have any follow up questions. 🐸 Jdlrobson (talk) 18:00, 18 October 2024 (UTC)

@Jdlrobson: this module is used a base for multiple other templates. Your change overrides derived template style declarations and caused for example Template:Zoroastrianism sidebar to display black-on-red text which made it difficult to read instead of white-on-red as it should. This module has a stylesheet. Does the color variable really need to be an inline style declaration instead of a line in .sidebar-list-title class? MKFI (talk) 08:10, 19 October 2024 (UTC)

Presumably it could be added via a class. When dark mode was rolled out, for templates I was less familiar with I usually went with the most straightforward solution. I see this one uses Module:Sidebar/styles.css ?

You should be able to see the exact issue it fixes (unreadable black text on a black background) by visiting https://en.wiki.x.io/wiki/Template:Sidebar/testcases?useskin=vector-2022&vectornightmode=1

and undoing that inline style using this code:

$('.sidebar-list-title[style]').each((a,i)=>i.setAttribute('style',i.getAttribute('style').replace('color: var(--color-base)', '')))

So you could likely move it to CSS, test if the issue is resolved and then undo my change. 🐸 Jdlrobson (talk) 16:21, 19 October 2024 (UTC)

@Jdlrobson: I can't do the change myself - the module and stylesheet are both template-protected. MKFI (talk) 08:33, 20 October 2024 (UTC)

Templatestyles parameter

@Izno: Are there still benefits of the caller templates using the numerous parameters |templatestyles=, |child templatestyles=, and |grandchild templatestyles= over a simple <templatestyles> in each? 172.97.141.219 (talk) 10:39, 30 November 2024 (UTC)

Yes. Using the dedicated parameter allows you to use the cascading part of CSS, namely that the CSS in the child will apply after, not before the core TemplateStyles. This avoids the case where the child's styles are potentially overridden by styles of the same specificity in the core styles. You should generally always use these parameters. Izno (talk) 06:03, 1 December 2024 (UTC)