Template talk:Side box

(Redirected from Template talk:Side box/testcases)
Latest comment: 2 years ago by Thumperward in topic Bullet points don't work

Plainlinks?

edit

I noticed that external links in the parameter |description= of the template {{Listen}} are not shown as such ( ). Examining that template showed that it uses {{Side box}} where class="plainlinks" is used. Is this behaviour deliberate or an unintended side effect? -- Michael Bednarek (talk) 05:49, 30 June 2010 (UTC)Reply

Definitely unintended. Can this be overridden? Chris Cunningham (not at work) - talk 12:30, 6 July 2010 (UTC)Reply
I don't know; why not simply remove the plainlinks class? -- Michael Bednarek (talk) 12:36, 6 July 2010 (UTC)Reply
The sisterlinks templates all use it. Truth be told, {{listen}} really shouldn't be subclassing here in the first place, as it's very different from the other templates that use it. Chris Cunningham (not at work) - talk 13:06, 6 July 2010 (UTC)Reply
That (sisterlinks) makes sense. I'm not technically savvy enough with templates to re-write {{Listen}}. Maybe plainlinks can be overridden with some kind of <div> or <span>? -- Michael Bednarek (talk) 13:45, 6 July 2010 (UTC)Reply

above parameter?

edit

When I was looking at {{Interwiki category}}, I wanted above parameter like below. It would be useful. Here's the sandbox. --fryed-peach (talk) 17:33, 9 August 2011 (UTC)Reply

I've massaged this into the current codebase, minus some additional changes that weren't present in the proposal. Chris Cunningham (user:thumperward) (talk) 15:26, 15 August 2012 (UTC)Reply

Edit request on 23 August 2013

edit

Replace

metadata

with

{{#ifeq:{{lc:{{{metadata|}}}}}|no||metadata}}

so this template can be used for content like it says on the tin. — Lfdder (talk) 11:16, 23 August 2013 (UTC)Reply

  Done Happymelon 11:36, 23 August 2013 (UTC)Reply
Thanks. Can you also look at the edit request here? — Lfdder (talk) 11:37, 23 August 2013 (UTC)Reply

Conversion to Lua

edit

I've written a Lua version of this template at Module:Side box. You can test it out using {{side box/sandbox}}, and there are test cases at Module:Side box/testcases (run) and Template:Side box/testcases. It should be a one-for-one switch, apart from the |metadata= parameter now accepting "no", "n", "false", and "0" to suppress the "metadata" class. This particular template isn't too resource-hungry, but it needs to be converted to Lua to allow some of its resource-hungry daughter templates, such as {{listen}}, to be converted as well. Does anyone have any objections to be updating the template? — Mr. Stradivarius ♪ talk ♪ 07:10, 2 June 2014 (UTC)Reply

I've gone ahead and converted it. Please let me know if you notice any odd behaviour from any side-box-related templates. — Mr. Stradivarius ♪ talk ♪ 05:01, 3 June 2014 (UTC)Reply

Protected edit request on 30 November 2014

edit

This template should have its own class. Currently, it uses mbox and mbox-text classes, which blocks setting the width:auto for any use case. --Rezonansowy (talk | contribs) 15:50, 30 November 2014 (UTC) Rezonansowy (talk | contribs) 15:50, 30 November 2014 (UTC)Reply

Please make your requested changes to the template's sandbox first; see WP:TESTCASES. Rezonansowy, what do you mean it "blocks setting the width:auto"? Something like this would need to be sandboxed before being carried out anyways, so let's work through it. — {{U|Technical 13}} (etc) 21:50, 30 November 2014 (UTC)Reply
@Technical 13: I'm not able to create a sandbox for a non-existing CSS class. Just create a separate CSS class for it and remove the width=100% property. --RezonansowyakaRezy (talk | contribs) 13:10, 24 January 2015 (UTC)Reply
  Not done for now: A class isn't needed for that. Where is the use case for where you need to set the width to auto? Simply add |style = width: auto !important and it will override the width: 100%; all by itself. If I'm misunderstanding you, then please show me where you want to use this feature and tell me why it is needed and I'll try to help you make it happen (even though this is a Lua module which I'm not very good with yet, I've set up a sandbox to play with and try and make needed adjustments). — {{U|Technical 13}} (etc) 16:42, 24 January 2015 (UTC)Reply

belowstyle? / "text"

edit

This template/module doesn't appear to offer a belowstyle parameter – is this correct/intentional? If so, any particular reason why..? If not intentional, may it be added..? Sardanaphalus (talk) 12:22, 15 January 2015 (UTC)Reply

PS As text isn't necessarily what might be placed in the middle of the box, could the text parameter be renamed content (or content made an alias for text, or...) and textstyle renamed contentstyle (or...), please..?


Working with tables

edit

It doesn't seem to be working with tables.  [ Derek Leung | LM ] 16:15, 13 May 2015 (UTC)Reply

Foreground color

edit

There's an issue with night mode skins where background is overwritten without changing text color leaving white text on white background. Please change in Module:Side box:

css{border = '1px solid #aaa', ['background-color'] = '#f9f9f9'}

to

css{border = '1px solid #aaa', ['background-color'] = '#f9f9f9', color = '#000'}

Dispenser 22:27, 16 February 2017 (UTC)Reply

  Done — Martin (MSGJ · talk) 14:37, 17 February 2017 (UTC)Reply

Fully protected edit request on 14 May 2017

edit

A protected redirect, Template:Sidebox, needs redirect category (rcat) templates added. Please modify it as follows:

  • from this:
#REDIRECT [[Template:Side box]]
  • to this:
#REDIRECT [[Template:Side box]]

{{Redirect category shell|
{{R from move}}
{{R from modification}}
{{R from template shortcut}}
}}
  • WHEN YOU COPY & PASTE, PLEASE LEAVE THE SKIPPED LINE BLANK FOR READABILITY.

The {{Redirect category shell}} template is used to sort redirects into one or more categories. When {{pp-protected}} and/or {{pp-move}} suffice, the Redirect category shell template will detect the protection level(s) and categorize the redirect automatically. (Also, the categories will be automatically removed or changed when and if protection is lifted, raised or lowered.) Thank you in advance!  Paine Ellsworth  put'r there  09:21, 14 May 2017 (UTC)Reply

  Not done: The page's protection level has changed since this request was placed. You should now be able to edit the page yourself. If you still seem to be unable to, please reopen the request with further details. Jo-Jo Eumerus (talk, contributions) 10:22, 14 May 2017 (UTC)Reply
Thank you very much, Jo-Jo Eumerus!  Paine Ellsworth  put'r there  10:44, 14 May 2017 (UTC)Reply

add the raw code to the documentation

edit

Hi, I suggest adding something like this to the documentation

{{Side box |metadata=No
 | above = '''''above''''' content appears here, in a separate row above the main text area
 | image = {{smaller|('''''image''''')}} 
 | text = '''''text''''' content, the box's main content, appears here
 | imageright = {{longitem|font-size:90%|('''''image<br/>right'''''{{hsp}})}}
 | below = '''''below''''' content appears here, in a separate row below the main text area
}}

to display which code generates the example box on the right. This will also make copy-pasting the code easy. AadaamS (talk) 18:05, 17 March 2018 (UTC)Reply

Template-protected edit request on 25 August 2018

edit

copies and adaptations you may only make copies or adaptations of the software for archival purposes or when copying or adaptation is an essential step in the authorized use of the software. you must you all copies or adaptations. you may not copy the software onto any public network. — Preceding unsigned comment added by 2604:2000:E8CC:700:D0B2:B41F:F1D7:BA99 (talk) 08:23, 25 August 2018 (UTC)Reply

Width seems slightly off

edit

Allow disabling plainlist with a parameter

edit

Hello,

At line 109 in Module:Side box there is the following code:
textCell:addClass('mbox-text plainlist')

Please can a parameter be added to disable the 'plainlist' class so that lists with bullet points or numbers can appear in this template?

Thanks, DesertPipeline (talk) 03:52, 26 June 2021 (UTC)Reply

@DesertPipeline: If nobody beats me to it, I can take a shot at this later today. – Rummskartoffel 09:19, 26 June 2021 (UTC)Reply
User:Rummskartoffel: Thank you very much :) DesertPipeline (talk) 09:27, 26 June 2021 (UTC)Reply
Please apply the changes made in the sandbox (diff, test case). This will add a parameter |plainlist= which, when set to "no", will suppress the "plainlist" class on the main text cell, allowing lists to be displayed normally. The parameter will default to "yes", i.e. the behaviour of existing uses of {{Side box}} will not change. For background, compare Wikipedia:Help desk#Bullet points not working within Template:Side box?.
After applying the change to the module, please also uncomment the new row for this parameter which I added to the table of parameters at Template:Side box/doc so that the documentation is up to date. – Rummskartoffel 20:25, 26 June 2021 (UTC)Reply
Rummskartoffel, do we know what the use case for this will be (i.e. why are we doing this?). Thanks! firefly ( t · c ) 08:25, 28 June 2021 (UTC)Reply
@Firefly: Since I just made this edit request in response to a help desk post, I don't have a specific use case myself (DesertPipeline, did you have something specific in mind?). But I don't see an obvious reason why it shouldn't be possible to have a normal-looking list in a side box – that is to say, I don't understand why this template has the "plainlist" class to begin with –, so this seems like a sensible thing to add. – Rummskartoffel 10:06, 28 June 2021 (UTC)Reply
Would it be better to add a |class=none parameter or similar which can override plainlist? Having a specific override parameter |plainlist= would be unnecessary then. This would be useful if you wanted e.g. a |class=hlist or something and avoids any need to add a further parameter later. User:GKFXtalk 16:27, 3 July 2021 (UTC)Reply
I'm not sure I'm understanding you correctly. Do you mean that |class=, when non-empty, would set the classes of the main text cell to only the classes specified (with "none" as a special case)? Or should it always keep mbox-text? Or something else? – Rummskartoffel 19:22, 3 July 2021 (UTC)Reply
User:Firefly: In my specific case the reason I need this is so that the article Trial of Socrates can have a box listing some Wikiquote documents with bullet points beside each one to delineate them better. Regards, DesertPipeline (talk) 11:21, 28 June 2021 (UTC)Reply
User:Firefly: Please can you implement the edit request? Thanks, DesertPipeline (talk) 04:52, 12 July 2021 (UTC)Reply
@DesertPipeline: I believe I have implemented effectively what you wanted. I also passed the new |textclass= parameter through {{Sister project}} and made an example by implementing it on Trial of Socrates (which seemed to be what you were trying to accomplish). —Uzume (talk) 07:51, 12 July 2021 (UTC)Reply
User:Uzume: Thank you very much! :) DesertPipeline (talk) 08:01, 12 July 2021 (UTC)Reply

bug: text flows on the right but not on the left

edit

If I place the box on the right of a page, the text flows around it, but if I place in on the left, it doesn't. — kwami (talk) 20:56, 10 October 2021 (UTC)Reply

bug: no image on mobile

edit

The page Crime in Sweden visited on mobile [1] displays the box, with the above and below texts, but not the image. --62.98.162.83 (talk) 07:50, 20 January 2022 (UTC)Reply

That's because this template uses mbox styling, and mbox-image is hidden on mobile. I've removed the template as totally unnecessary. Izno (talk) 20:24, 27 May 2022 (UTC)Reply

New edit breaks {{algebraic notation}}

edit

I'm not sure how to fix this but this edit broke {{algebraic notation}}. When used with |pos=toc, it used to look like this, in a box next to the TOC. Now it's just vertical text. But it doesn't stop there; on desktop if you zoom in enough (Ctrl + +), it goes back to normal. ~~ lol1VNIO⁠🎌 (talk • contribs) 16:18, 10 July 2022 (UTC)Reply

This will be a non-issue in new Vector (e.g. this view), and even in current Vector this is not unusable (in Timeless it gets hidden behind the table of contents, isn't that nice? :).
What's different now is that a table is a little more wrappier than a div. I will add some templatestyles for the algebraic notation template to restore the older behavior.
Regarding "old behavior", that's a coincidence. The reason it does so is different than the older cause. Izno (talk) 17:51, 10 July 2022 (UTC)Reply
Should be fixed now, +- job queue updates. Null edit a page to see differences. Izno (talk) 18:07, 10 July 2022 (UTC)Reply
Thanks! ~~ lol1VNIO⁠🎌 (talk • contribs) 21:03, 10 July 2022 (UTC)Reply

Bullet points don't work

edit

Bullet points in the "text" field don't seem to work. HLHJ (talk) 23:53, 29 October 2022 (UTC)Reply

@HLHJ: This is because the template adds the "plainlist" CSS class by default, which removes the bullet points from lists. You can work around this by specifying a dummy class to the |textclass= parameter. For example,
{{side box|textclass=vlist|text=
Some text
* foo
* bar
}}
will produce the side bar below:
Mr. Stradivarius ♪ talk ♪ 00:49, 30 October 2022 (UTC)Reply
Thank you, Mr. Stradivarius! I'll do that. I assume this behaviour is useful to others.  . HLHJ (talk) 00:55, 30 October 2022 (UTC)Reply
The plainlist class was originally added back in 2012 by Thumperward. His edit summary said that the plainlist class "adds semantic value when multiple elements are used", but I'm not sure exactly why that would be. If we wanted, we could change the default, but we would need to check all of the transclusions to see which ones would need fixing, which would be a pain. — Mr. Stradivarius ♪ talk ♪ 01:41, 30 October 2022 (UTC)Reply
Thank you again. Thumperward may have useful comments; I really have none! Happy with whatever works best for others, it's not a big deal for me. HLHJ (talk) 02:16, 30 October 2022 (UTC)Reply
Hmm. Does not work flawlessly if the first line is bulleted. HLHJ (talk) 02:41, 30 October 2022 (UTC)Reply
You can work around that problem with an HTML-encoded space:
{{side box|textclass=vlist|text=&#32;
* Some text
* foo
* bar
}}
This is because MediaWiki trims the whitespace at the start of the text, so the newline character disappears. By adding the HTML-encoded space, the newline character is included. (It is hardly elegant, however.) The output looks like this:
Mr. Stradivarius ♪ talk ♪ 08:40, 30 October 2022 (UTC)Reply

I think changing the default after a decade is likely to have a somewhat deleterious effect considering that there are presently 1038789 transclusions of this template. Apologies if I'm a little out of the loop here as I haven't thought about this template in years, but the test cases page suggests that a year ago Rummskartoffel (talk · contribs) introduced a test case for a plainlist=no parameter. This doesn't ever seem to have actually been implemented in the code. I think that's probably the right approach if someone wants to look into it. If you need help please feel free to ping me. Chris Cunningham (user:thumperward) (talk) 12:25, 30 October 2022 (UTC)Reply