Moduldiskussion:Rollista

Larske, Innocent bystander: Det här är mitt första Lua-försök. Det är till Mall:Rollista. Tanken är att ta bort den upprepade koden som just nu finns i Mall:Rollista och tillåta godtyckligt många rader. Min testmall finns på Användare:Nirmos/Rollista. Användare:Nirmos/RollistaTest använder Mall:Rollista och Användare:Nirmos/RollistaTestLua använder Användare:Nirmos/Rollista. Eftersom Mall:Rollista används på ett par tusen sidor är det nog bäst att ni kollar igenom så att allt ser bra ut innan det byts ut. Nirmos (diskussion) 4 januari 2017 kl. 01.45 (CET)

Det finns många mallar som är väldigt oöverskådliga för att man har stoppat in en lång uppräkning av parametrar. Och hur lång den än är, är det alltid någon som vill ha den ännu längre. Mallen {{Rollista}} är ett bra exempel även om det verkar som om den inte är så mycket använd. (Jämför användningen av mallen med antalet artiklar som har ett avsnitt för Rollista). Det kan möjligen bero på att man hellre vill se listor som punktlistor snarare än som tabeller som det blir med den här mallen/modulen. Men det är en annan fråga.

Koden ser bra ut tycker jag. Den är en ersättare till den komplicerade wikikoden och kan därför sjösättas i mallen. Möjligen skulle man komplettera med ett felutskrift om antalet parametrar inte är jämnt, nu blir det inget resultat alls om man skulle missa att ange en parameter. Så här kanske (mellan rad 25 och rad 26):

else
   return pframe:preprocess('{{error|Anropet innehåller inte ett jämnt antal parametrar!}}')

--Larske (diskussion) 4 januari 2017 kl. 05.57 (CET)

Ger inte #pframe.args samma sak som "n" ger dig här? Kanske inte, om det finns även namngivna parametrar?

Jag hade nog använt html och inte wikikod här. Stödet för transkluderad wikikod från en modul är svagt.

Man kan använda funktioner som mw.text.tag('td', {style="padding:0 4px;width:5em", align="right", valign="top"}, 'Blabla')

En par av 'td' läggs inom 'tr' och en samling 'tr' läggs inom 'table' - et wipps, så har man en tabell. -- Innocent bystander (diskussion) 4 januari 2017 kl. 09.07 (CET)

Visar vad jag menar i Modul:Sandlådan/Innocent bystander/2. Får dock inte felmeddelandet om "ojämnt antal parametrar" att gå igenom som jag tänkte. -- Innocent bystander (diskussion) 4 januari 2017 kl. 09.42 (CET)

Det stämmer att man tyvärr inte kan komma åt antalet poster i en Lua-tabell med "#". --Larske (diskussion) 4 januari 2017 kl. 10.37 (CET)

Nu tror jag att jag fick det att fungera. En sak: Den här background-color:#f5f5f5 ger inga synliga effekter alls på min dator och jag har då testat även Chrome! -- Innocent bystander (diskussion) 4 januari 2017 kl. 11.20 (CET)

Och som alltid, vill någon "låna" kod jag har författat ihop, så är det fritt fram att göra så utan attributeringar och annat trams! -- Innocent bystander (diskussion) 4 januari 2017 kl. 12.11 (CET)

Larske: Jag gillar idén med felmeddelande. Faktum är att jag gillar den så mycket att jag skulle vilja ta ett steg längre och lägga till en kategori, ungefär så här. Är det ok? Då skulle vi till och med kunna upptäcka sidor där det redan är fel, om det finns sådana sidor (inte omedelbart naturligtvis, men när de parsas om).

Innocent bystander: Tack för att du förenklade det jag skrev, jag tror vi kör med din version. Vad jag kan se läggs bakgrundfärgen till korrekt. Problemet är nog snarare att den färgen är så svag att du inte ser den. Om du ändrar till röd så ser du att det fungerar även med din version. Nirmos (diskussion) 5 januari 2017 kl. 01.53 (CET)

Jag kollade bakgrundsfärgen genom att titta på html-koden i sidor jag testade att lägga in koden i. Det såg rätt ut där, så... -- Innocent bystander (diskussion) 5 januari 2017 kl. 08.50 (CET)