Template talk:Series overview
Template:Series overview is permanently protected from editing because it is a heavily used or highly visible template. 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 template-protected}} to notify an administrator or template editor to make the requested edit. Usually, any contributor may edit the template's documentation to add usage notes or categories.
Any contributor may edit the template's sandbox. Functionality of the template can be checked using test cases. |
This is the talk page for discussing improvements to the Series overview template. |
|
Archives: 1, 2, 3Auto-archiving period: 30 days |
Television Template‑class | |||||||
|
Unknown parameters check
editThis edit request has been answered. Set the |answered= or |ans= parameter to no to reactivate your request. |
Please add a module to check unknown parameters to this template. I made a list of parameters + regular expressions for all cases, which should cover ~98% of cases. I have already checked this list in Russian wikipedia and fixed about a dozen errors. There may be some unusual cases that will also need to be added to the exceptions or otherwise corrected in the articles. (Perhaps the list of expressions itself could be simplified somewhere).
{{#invoke:check for unknown parameters|check | unknown = {{main other|[[Category:Pages using series overview with unknown parameters|_VALUE_{{PAGENAME}}]]}} | ignoreblank = | preview = Page using [[Template:Series overview]] with unknown parameter "_VALUE_" | showblankpositional = 1 | allreleased | caption | country | dontclose | id | multiseries | network | network_transclude | released | seasonT | series | seriesT | width | regexp1 = aux[%u] | regexp2 = aux[%u][%d]+[%u]? | regexp3 = color[%d]+S?[%u]? | regexp4 = end[%d]+S?[%u]? | regexp5 = episodes[%d]+S?[%u]? | regexp6 = info[%u] | regexp7 = info[%u][%d]+S? | regexp8 = info[%u]_transclude | regexp9 = linkR[%d]+S? | regexp10 = linkT[%d]+S?[%u]? | regexp11 = link[%d]+S?[%u]? | regexp12 = network[%d]+[%u]? | regexp13 = postaux[%u][%d]+[%u]? | regexp14 = postaux[%u] | regexp15 = start[%d]+S?[%u]? | regexp16 = aux[%u][%d]+[%u]?span | regexp17 = info[%u][%d]+span | regexp18 = info[%u]span[%d]+S | regexp19 = network[%d]+[%u]?span | regexp20 = postaux[%u][%d]+[%u]?span }}
Solidest (talk) 23:13, 4 March 2024 (UTC)
- Added. Gonnym (talk) 06:07, 5 March 2024 (UTC)
- Thanks! I'll clean up some of them and add to the list later on. Solidest (talk) 11:24, 5 March 2024 (UTC)
|part=
(with number and special ability) needs to be added to this. - Favre1fan93 (talk) 21:17, 5 March 2024 (UTC)- Also some stuff with
|network=
with numbers, "S" and letters. See List of The Office (British TV series) episodes as an example. - Favre1fan93 (talk) 21:47, 5 March 2024 (UTC)- Thanks, I've been collecting all the cases too. So here's the final update, 7 & 12 to replace, and 21 to add:
- Thanks! I'll clean up some of them and add to the list later on. Solidest (talk) 11:24, 5 March 2024 (UTC)
| regexp7 = info[%u][%d]+S?[%u]? | regexp12 = network[%d]+S?[%u]? | regex21 = part[%d]+S?
- The only problem left is with the Talking Tom & Friends article, which I think needs to be fixed in the module code rather than added to the tracking. I will write about it a bit later. Solidest (talk) 00:15, 6 March 2024 (UTC)
- Added fixes to template. Gonnym (talk) 08:58, 6 March 2024 (UTC)
- The only problem left is with the Talking Tom & Friends article, which I think needs to be fixed in the module code rather than added to the tracking. I will write about it a bit later. Solidest (talk) 00:15, 6 March 2024 (UTC)
Strange triggering of invalid parameters
editIn some cases, parameters with a dot at the end show results instead of being treated as an unknown param: 1, 2. All of these cases displayed the data in the table. And also "episode2s" treated and worked here as "episodes2": 3 . Solidest (talk) 00:32, 6 March 2024 (UTC)
- The code here (line 136) is the problem it seems:
local str, num, str2 = string.match(k, '([^%d]*)(%d*)(%a*)')
- This ignores the period at the end.
- A testcase:
k = "infoB3." local str, num, str2 = string.match(k, '([^%d]*)(%d*)(%a*)') print(str) print(num) print(str2)
- prints "infoB" and "3". "str2" is empty here.
- Then at line 149 it adds the variable it found (without the period) to the new list.
- Gonnym (talk) 09:28, 6 March 2024 (UTC)
SeasonEntries[num][str .. str2] = v
Accessibility Issues
editHi, I currently have an episode list at WP:FLC. A reviewer brought up the fact that this template isn't screen reader friendly because it uses "col" rather than "colgroup" in the headers. Is this something that can be easily fixed? TheDoctorWho (talk) 19:58, 15 June 2024 (UTC)
- As stated at the FLC nomination, this is hardcoded into the template that is used on over 8,000 articles, multiple of which have passed FL. I see no reason why this needs to change. Tagging MPGuy2824. -- Alex_21 TALK 00:46, 16 June 2024 (UTC)
- No comment on the FLC nom, but I don't see any issue with adjusting this. It shouldn't alter any other instance, and would be an overall positive I believe for the template. Nothing wrong with updating to meet proper accessibility standards/guidelines. - Favre1fan93 (talk) 15:30, 16 June 2024 (UTC)
Template-protected edit request on 24 June 2024
editThis edit request to Module:Series overview has been answered. Set the |answered= or |ans= parameter to no to reactivate your request. |
Please do the changes changes made to this module's sandbox to the main module. This is more right from an accessibility perspective. -MPGuy2824 (talk) 07:45, 24 June 2024 (UTC)
- Done. @TheDoctorWho: this should be address now if you need to go back to your FLC about it. - Favre1fan93 (talk) 17:43, 24 June 2024 (UTC)