Template talk:Series overview

Mobile view

Is there a reason why this template look so bad on mobile view? It is no longer scrollable from left to right like a normal wikitable but it just go out of bounds of the screen. Lulusword (talk) 07:37, 18 December 2022 (UTC)

@Lulusword Sorry, I didn't come across this message before you posted it at WT:LUA, so I'll make any further updates to the issue here once I determine the issue. Cheers. -- Alex_21 _TALK 05:01, 31 December 2022 (UTC)

The problem should now be fixed. -- Alex_21 _TALK 06:17, 31 December 2022 (UTC)

@Alex 21: Thank you! Lulusword (talk) 07:13, 31 December 2022 (UTC)

Alex 21: These changes appear to have caused new Linter errors in articles including Stargate and Outline of the Marvel Cinematic Universe. They show as missing </div> tags, but sometimes the Linter error-detection gets confused by complex table syntax. – Jonesey95 (talk) 07:35, 31 December 2022 (UTC)

Hey, this also caused the header row to move down and hide the first row for all uses of the template on my screen https://ibb.co/Vv3HQT7. Thanks, Indagate (talk) 11:58, 31 December 2022 (UTC)

Hi! Could we fix this? I had no idea that the table reaching the end of the page was not on purpose. It is much easier to access as a scroll when on mobile devices, thanks. - SecondLooneyaccount (talk) — Preceding undated comment added 04:18, 2 January 2023‎ (UTC)

Still working on this. I know what the issue is, just not how to solve it. I'll keep you all updated. -- Alex_21 _TALK 21:35, 9 January 2023 (UTC)

 Done Sorry for the delay, but this should be completely fixed, I've tested it to the best of my ability. Long story short, I've surrounded all series overview tables with a <div>...</div> tag that wraps the content properly. For multi-series tables that used {{End}}, I've had to update those to use the new {{Series overview/end}}, which includes closing </div> and </table> tags; {{End}} only adds/added </table>. -- Alex_21 _TALK 08:05, 22 February 2023 (UTC)

@Alex 21 Caused same issue as last time, please see screenshot https://ibb.co/99Dckjd. Thanks, Indagate (talk) 08:19, 22 February 2023 (UTC)

@Indagate What browser are you using? I've cleared the browser- and site-caches on both Chrome and Firefox, as well as my phone's browser, and can't seem to replicate this issue. -- Alex_21 _TALK 08:22, 22 February 2023 (UTC)

Ah, seems working fine now. Cleared cache and tried incog this morning but had issue, but worked now. Thanks, Indagate (talk) 12:49, 22 February 2023 (UTC)

I am having the same issue when enabling the Sticky Table Header gadget. Other tables, for example Template:Episode_list/sublist/testcases, doesn't have any issue. I couldn't figure out what is the different between those two tables. Ngoclong19 (talk) 20:31, 3 April 2023 (UTC)

I've figured out why this template doesn't work with Sticky Table Header gadget. The table header doesn't stick, and it's offset from the table top. See screenshot in Indagate's comment above.

The surrounding div has overflow-x: auto, which causes issue with the table header's position: sticky. See here.

It looks like there's no native CSS solution. Ngoclong19 (talk) 14:21, 5 March 2024 (UTC)

@Alex 21: Looks like this again causes linter errors, on pages that use multiseries (for example List of Marvel Cinematic Universe television series). I've got a minimally reproducible example in my sandbox. --rchard2scout (talk) 12:53, 22 February 2023 (UTC)

@Rchard2scout I've made an update based on what I believe may be the error. If it's not the cause, then I feel the issue may be caused by one template generating the opening <div>, and then a separate template generating the closing </div>, which is not an situation that can be rectified. -- Alex_21 _TALK 21:36, 22 February 2023 (UTC)

Update on this, my changes today seem to have worked. Please correct me if I'm wrong. -- Alex_21 _TALK 01:12, 23 February 2023 (UTC)

Yes, it looks like it worked. Thanks! --rchard2scout (talk) 23:33, 23 February 2023 (UTC)

Unknown parameters check

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:

| 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)

Strange triggering of invalid parameters

In 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.

SeasonEntries[num][str .. str2] = v

Gonnym (talk) 09:28, 6 March 2024 (UTC)

Standardization of the order in which suffixes are added (or specials/parts order bug)

The module is now experiencing inconsistent triggering of suffixes. In some cases it works in any order (network1A = networkA1). And in some cases on the contrary, the standard formats described in the documentation do not work (see examples below). In order to maintain logic and obviousness, I propose to bring the order of adding suffixes to a single form in the module code:

[param][letter of params info/aux/postaux][season number][special][letter of part][span]

Currently there are problems with using parts of seasons and specials letter at the same time. See example: Talking Tom & Friends#Talking Tom Shorts

• [param][letter of part][season number][special] = startA1S is working
• [param][season number][letter of part][special] = start1AS is not working - table is breaking and outputting the text "1SA" directly into the table.

However, in all articles at the moment, the season number takes precedence, and [letter of part] is nowhere listed prior to [season number]. In the same article, start2B and start3S are listed next. And the "start2SA" option is also a standard: example here.

Also at the moment there are problems with "span" working simultaneously with specials. It doesn't work if it is placed at the end of a param. See List of Star Wars television series#Animated series.

• [param][letter of param][span][season number][special] = infoAspan0S is working
• [param][letter of param][season number][special][span] = infoA0Sspan is not working.

While all the other possible "span" writing variants are working well with "span" being placed at the end.

It seems illogical to me that inconsistency takes place, and it rather feels like a bug that should be fixed, instead of putting all these exceptions and nuances in the documentation. But don't know if this would be possible/easy to fix or not in the current implementation of the module. (Fixing this would also reduce duplication in the list of known parameters listed above, and save from adding 5+ variants which are only found in the Talking Tom article.) Solidest (talk) 00:55, 6 March 2024 (UTC)