Module talk:Message box
This module was considered for deletion on 2020 February 15. The result of the discussion was "keep". |
Module:Message box is permanently protected from editing because it is a heavily used or highly visible module. Substantial changes should first be proposed and discussed here on this page. If the proposal is uncontroversial or has been discussed and is supported by consensus, editors may use {{edit protected}} to notify an administrator to make the requested edit.
|
Text and/or other creative content from this version of Module:Message_box was copied or moved into incubator:Module:Wp/nod/Message_box with this edit. The former page's history now serves to provide attribution for that content in the latter page, and it must not be deleted as long as the latter page exists. |
|
|
This page has archives. Sections older than 30 days may be automatically archived by Lowercase sigmabot III when more than 4 sections are present. |
Edit request 23 November 2023[edit]
This edit request has been answered. Set the |answered= or |ans= parameter to no to reactivate your request. |
On Module:Message box/configuration, after line 152, please add
license-related = { class = 'imbox-license', image = 'Imbox-license.svg' },
.
Reasoning: Some templates, like {{Insignia}}, use the "license" type of imbox. But this produces a licensetpl class, which confuses whatever MediaWiki thing populates Category:Files with no machine-readable license. Adding this would allow those to be converted to an identical-looking type "license-related" without the class, depopulating much of the category.
Best, — Mdaniels5757 (talk • contribs) 20:47, 23 November 2023 (UTC)
- Done * Pppery * it has begun... 02:42, 24 November 2023 (UTC)
Edit request to alter removalNotice text[edit]
This edit request to Module:Message box/configuration has been answered. Set the |answered= or |ans= parameter to no to reactivate your request. |
The removal notice in message boxes refers to the displayed banner as a "template message". That is overly described; nobody cares how it was produced (if they did, maybe we should call it a "module message", or a "template-driven, module-implemented message"); they only care what it is you want to remove (and how). Suggested improvement:
− | removalNotice = '<small>[[Help:Maintenance template removal|Learn how and when to remove this | + | removalNotice = '<small>[[Help:Maintenance template removal|Learn how and when to remove this message]]</small>', |
To me, it's a "banner", not a "message", so I'd prefer the former word, but I wouldn't object to just "message" without the "template" part. Among other things, the shorter version will cause a fraction of banners to have one fewer line at certain page widths. Thanks, Mathglot (talk) 22:26, 27 April 2024 (UTC)
- Done * Pppery * it has begun... 03:25, 29 April 2024 (UTC)
Protected edit request on 4 May 2024[edit]
This edit request to Module:Message box/ombox.css has been answered. Set the |answered= or |ans= parameter to no to reactivate your request. |
This ombox defines a backgroundcolor, without defining a text color, which is problematic in the new darkmod. I suggest we add color: #202122
to the first styling block to. —TheDJ (talk • contribs) 15:58, 4 May 2024 (UTC)
Edit request 8 May 2024[edit]
This edit request to Module:Message box/ombox.css has been answered. Set the |answered= or |ans= parameter to no to reactivate your request. |
Description of suggested change: Add CSS styling for night mode to the ombox styles.css. Diff:
− | .ombox {
margin: 4px 0;
border-collapse: collapse;
border: 1px solid #a2a9b1; /* Default "notice" gray */
background-color: #f8f9fa;
box-sizing: border-box; | + | .ombox {
margin: 4px 0; border-collapse: collapse; border: 1px solid #a2a9b1; /* Default "notice" gray */ background-color: #f8f9fa; box-sizing: border-box; } html.skin-theme-clientpref-night .ombox { margin: 4px 0; border-collapse: collapse; border: 1px solid #f8f9fa; /* Off-white */ background-color: #00143d; /* Dark blue */ box-sizing: border-box; } @media (prefers-color-scheme: dark) { html.skin-theme-clientpref-night .ombox { margin: 4px 0; border-collapse: collapse; border: 1px solid #f8f9fa; /* Off-white */ background-color: #00143d; /* Dark blue */ box-sizing: border-box; } } /* For the "small=yes" option. */ .ombox.mbox-small { font-size: 88%; line-height: 1.25em; } .ombox-speedy { border: 2px solid #b32424; /* Red */ background-color: #fee7e6; /* Pink */ } html.skin-theme-clientpref-night .ombox-speedy { border: 2px solid #ffdbdb; /* Light pink */ background-color: #b32424; /* Red */ } @media (prefers-color-scheme: dark) { html.skin-theme-clientpref-night .ombox-speedy { border: 2px solid #ffdbdb; /* Light pink */ background-color: #b32424; /* Red */ } } .ombox-delete { border: 2px solid #b32424; /* Red */ } html.skin-theme-clientpref-night .ombox-delete { border: 2px solid #ff6961; /* Pink */ background-color: #b32424; /* Red */ } @media (prefers-color-scheme: dark) { html.skin-theme-clientpref-night .ombox-delete { border: 2px solid #ff6961; /* Pink */ background-color: #b32424; /* Red */ } } .ombox-content { border: 1px solid #f28500; /* Orange */ } html.skin-theme-clientpref-night .ombox-content { border: 1px solid #ffe7ce; /* Off-white */ background-color: #ff8f05; /* Orange */ } @media (prefers-color-scheme: dark) { html.skin-theme-clientpref-night .ombox-content { border: 1px solid #ffe7ce; /* Off-white */ background-color: #ff8f05; /* Orange */ } } .ombox-style { border: 1px solid #fc3; /* Yellow */ } html.skin-theme-clientpref-night .ombox-style { border: 1px solid #fff9db; /* Off-white */ background-color: #fad000; /* Yellow */ } @media (prefers-color-scheme: dark) { html.skin-theme-clientpref-night .ombox-style { border: 1px solid #fff9db; /* Off-white */ background-color: #fad000; /* Yellow */ } } .ombox-move { border: 1px solid #9932cc; /* Purple */ } html.skin-theme-clientpref-night .ombox-move { border: 1px solid #c9b3ff; /* Light purple */ background-color: #7500db; /* Purple */ } @media (prefers-color-scheme: dark) { html.skin-theme-clientpref-night .ombox-move { border: 1px solid #c9b3ff; /* Light purple */ background-color: #7500db; /* Purple */ } } .ombox-protection { border: 2px solid #a2a9b1; /* Gray-gold */ } html.skin-theme-clientpref-night .ombox-protection { border: 1px solid #fff; /* White */ background-color: #a2a9b1; /* Blueish-light gray */ } @media (prefers-color-scheme: dark) { html.skin-theme-clientpref-night .ombox-protection { border: 1px solid #fff; /* White */ background-color: #a2a9b1; /* Blueish-light gray */ } } |
Andumé (talk) 23:45, 8 May 2024 (UTC)
- Another option with a darker/higher contrast/more consistent color scheme would be this:
− | .ombox {
margin: 4px 0; border-collapse: collapse; border: 1px solid #a2a9b1; /* Default "notice" gray */ background-color: #f8f9fa; box-sizing: border-box; | + | .ombox {
margin: 4px 0; border-collapse: collapse; border: 1px solid #a2a9b1; /* Default "notice" gray */ background-color: #f8f9fa; box-sizing: border-box; } html.skin-theme-clientpref-night .ombox { margin: 4px 0; border-collapse: collapse; border: 1px solid #f8f9fa; /* Off-white */ background-color: #00143d; /* Dark blue */ box-sizing: border-box; } @media (prefers-color-scheme: dark) { html.skin-theme-clientpref-night .ombox { margin: 4px 0; border-collapse: collapse; border: 1px solid #f8f9fa; /* Off-white */ background-color: #00143d; /* Dark blue */ box-sizing: border-box; } } /* For the "small=yes" option. */ .ombox.mbox-small { font-size: 88%; line-height: 1.25em; } .ombox-speedy { border: 2px solid #b32424; /* Red */ background-color: #fee7e6; /* Pink */ } html.skin-theme-clientpref-night .ombox-speedy { border: 2px solid #ffdbdb; /* Light pink */ background-color: #571818; /* Dark red */ } @media (prefers-color-scheme: dark) { html.skin-theme-clientpref-night .ombox-speedy { border: 2px solid #ffdbdb; /* Light pink */ background-color: #571818; /* Dark red */ } } .ombox-delete { border: 2px solid #b32424; /* Red */ } html.skin-theme-clientpref-night .ombox-delete { border: 2px solid #ff6961; /* Pink */ background-color: #571818; /* Dark red */ } @media (prefers-color-scheme: dark) { html.skin-theme-clientpref-night .ombox-delete { border: 2px solid #ff6961; /* Pink */ background-color: #571818; /* Dark red */ } } .ombox-content { border: 1px solid #f28500; /* Orange */ } html.skin-theme-clientpref-night .ombox-content { border: 1px solid #ffe7ce; /* Off-white */ background-color: #955200; /* Dark orange */ } @media (prefers-color-scheme: dark) { html.skin-theme-clientpref-night .ombox-content { border: 1px solid #ffe7ce; /* Off-white */ background-color: #955200; /* Dark orange */ } } .ombox-style { border: 1px solid #fc3; /* Yellow */ } html.skin-theme-clientpref-night .ombox-style { border: 1px solid #fff9db; /* Off-white */ background-color: #9d7900; /* Dark yellow */ } @media (prefers-color-scheme: dark) { html.skin-theme-clientpref-night .ombox-style { border: 1px solid #fff9db; /* Off-white */ background-color: #9d7900; /* Dark yellow */ } } .ombox-move { border: 1px solid #9932cc; /* Purple */ } html.skin-theme-clientpref-night .ombox-move { border: 1px solid #c9b3ff; /* Light purple */ background-color: #2d0055; /* Dark purple */ } @media (prefers-color-scheme: dark) { html.skin-theme-clientpref-night .ombox-move { border: 1px solid #c9b3ff; /* Light purple */ background-color: #2d0055; /* Dark purple */ } } .ombox-protection { border: 2px solid #a2a9b1; /* Blue-gray */ } html.skin-theme-clientpref-night .ombox-protection { border: 1px solid #fff; /* White */ background-color: #3b3d40; /* Blueish-dark gray */ } @media (prefers-color-scheme: dark) { html.skin-theme-clientpref-night .ombox-protection { border: 1px solid #fff; /* White */ background-color: #3b3d40; /* Blueish-dark gray */ } } |
- Andumé (talk) 17:37, 11 May 2024 (UTC)
- Not done: These diffs introduce duplicate styles and more color styles than are necessary to deal with the issue of dark mode. Thanks for throwing something up though, I will take a look. Izno (talk) 04:06, 21 May 2024 (UTC)
- @I Am Andumé For the first, you copy-pasted the full .ombox definition in the context of the color CSS, which is surely not intended. For the second, you typoed the prefers-color-scheme dark selector (it should be -os, not -night), made multiple separate blocks for the prefers-color scheme (perhaps to show that you were consistent between -dark and -os settings, IDK), and more concerningly you strayed to changing the background rather than changing just the border. We should do something which matches the earlier rules in what is being set, which for most of these cases is just the border. Izno (talk) 04:22, 21 May 2024 (UTC)
- Actually taking a look what happens when the background is dark, I don't think we need to change the border colors at all and can leave those the same as the base colors. That simplifies maintenance tremendously. We need to pick a good dark color for the typical background, as well as the speedy color. Izno (talk) 04:37, 21 May 2024 (UTC)
- I put those in the official sandbox and loaded that so now you can see them on Template:Ombox/testcases#name=_text=text. Izno (talk) 04:55, 21 May 2024 (UTC)
- @Izno:
- Thanks for taking a look at my request. Here are a few things I'd like to mention:
- The reason I copy-pasted the full .ombox definition, instead of just the styling that needed to be different in night mode was because I was afraid the other CSS would not be applied to the night mode version of the template if I omitted it.
- That was a typo, yes.
- I made separate block for the prefers-color-scheme as I was not yet aware it was possible to combine it all into one.
- Changing the background color was necessary as otherwise it would have been very bright (as you seem to have noticed). Also, without either defining color or changing background color, the text would not have been readable.
- Anyways, I wrote this CSS quite a while ago, so I was very inexperienced at the time, which explains a lot of the mistakes I made. Andumé (talk) 05:07, 21 May 2024 (UTC)
- Actually taking a look what happens when the background is dark, I don't think we need to change the border colors at all and can leave those the same as the base colors. That simplifies maintenance tremendously. We need to pick a good dark color for the typical background, as well as the speedy color. Izno (talk) 04:37, 21 May 2024 (UTC)
- @I Am Andumé For the first, you copy-pasted the full .ombox definition in the context of the color CSS, which is surely not intended. For the second, you typoed the prefers-color-scheme dark selector (it should be -os, not -night), made multiple separate blocks for the prefers-color scheme (perhaps to show that you were consistent between -dark and -os settings, IDK), and more concerningly you strayed to changing the background rather than changing just the border. We should do something which matches the earlier rules in what is being set, which for most of these cases is just the border. Izno (talk) 04:22, 21 May 2024 (UTC)