MediaWiki-diskussion:Gadget-SignUp.js

Tar bort

Sara Mörtsell (WMSE), Ainali, Josve05a: Jag tar bort det här. Jag uppfattade det först som att den här finessen skulle användas av alla på varje skrivstuga, i vilket fall den kanske skulle kunna ses som berättigad. Jag förstår nu att användare skriver upp sig snarare en gång per säsong. Vi måste väga nyttan mot att besökare måste ladda ned och utföra koden. Det finns mycket mer användbara saker vi kan göra med den här mängden kod. Nirmos (diskussion) 29 september 2016 kl. 18.47 (CEST)

Jaha, jag tänkte använda den på skrivstugan i Visby om ett par veckor, men har inte hunnit skapa den undersidan än. ♥Ainali ^diskussion_bidrag 29 september 2016 kl. 21.12 (CEST)

‎Begäran om redigering: definierbar plats för knapp

Ersätt raderna 9 - 13 med:

$(".gadget-signup-placeholder").replaceWith(button);

Då kan man lägga till elementet

<span class="gadget-signup-placeholder"></span>

på den plats i wikitexten där man vill att knappen ska hamna. /Sebastian Berlin (WMSE) (diskussion) 6 mars 2018 kl. 14.16 (CET)

För det första är det inte självklart att en sådan ändring skulle vara önskad av alla som kan tänkas vilja använda den här knappen (det är en ganska stor ändring designmässigt). Om det här görs över huvud taget bör det vara frivilligt och falla tillbaka på det gamla beteendet om det inte finns någon placeholder. Det är en stor skillnad på att kräva att knappen ska läggas till manuellt och att lägga till möjligheten att göra det.

För det andra skulle det inte gå eftersom knappens position i förhållande till .mw-editsection används för att lista ut vilken sektion den ligger i. Mer specifikt blir det när

var section = href.match( /section=(\d+)/ )[ 1 ];

försöker utföras. Testa gärna att det fungerar i Special:Min sida/common.js eller i webbläsarkonsolen först. Nirmos (diskussion) 6 mars 2018 kl. 16.38 (CET)

Det låter rimligt att ha kvar det gamla beteendet; jag har lagt till det nya sättet parallellt. Jag har också uppdaterat logiken för att hitta rätt sektion att lägga till signaturerna i, så att båda sortens knappar fungerar.

Den nya koden bor på Special:Omdirigering/revision/42562925. Det ända som behöver ändras är att ta bort true || i if-satsen på rad 8. Jag behövde denna när jag labbade, vilket jag gjorde på Användare:Sebastian_Berlin_(WMSE)/Singup_Gadget_Test. Tack för tipset om att använda common.js förresten. Jag jobbade tidigare med koden i JSFiddle, varför det slank igenom några missar. /Sebastian Berlin (WMSE) (diskussion) 7 mars 2018 kl. 10.52 (CET)

Sebastian Berlin (WMSE): Jag har till stor del infört ditt förslag, med några ändringar:

1. I stället för att både lägga till knappen på de förutbestämda platserna och de automatiska (vilket din kod gör) så läggs ingen knapp till automatiskt i min version om det finns en placeholder. Den som lägger till en knapp manuellt vill troligtvis inte ha de automatiska också.
2. Extra if-sats för href innan någon åtkomst försöker göras till den
3. Begränsar till action=view eftersom det inte finns några sektionsredigeringslänkar i redigeringsläget
4. Ändrat CSS:en i MediaWiki:Gadget-SignUp.css så att marginalen enbart läggs till när knappen läggs till automatiskt. Det ger den som lägger till knappen manuellt större kontroll över hur den ser ut.

Nirmos (diskussion) 7 mars 2018 kl. 18.46 (CET)

Bra. Jag testade att lägga till den där det var tänkt att den skulle vara och allt verkar funka som det ska. Tack för hjälpen. /Sebastian Berlin (WMSE) (diskussion) 7 mars 2018 kl. 18.59 (CET)

Sara Mörtsell (WMSE): De här ändringarna bör vara live nu om du vill välja var knappen ska vara någonstans. Nirmos (diskussion) 7 mars 2018 kl. 18.55 (CET)

Ser toppen ut. Tack för hjälpen. --Sara Mörtsell (WMSE) (diskussion) 7 mars 2018 kl. 19.54 (CET)