Discussion MediaWiki:Gadget-CommonEdit.js

Remove redundant check

Hi!

I think we can change "forcePreview" code from

  if (wgUserName != null || wgAction != "edit") return;

to

  if (wgUserName != null) return;

because MediaWiki:Common.js/edit.js is only loaded when edit a page. (Or this would cause a problem with "|| wgAction == 'submit'" at MediaWiki:Common.js?) Heldergeovane (d) 20 juillet 2009 à 14:11 (CEST)

I'm not sure to understand: MediaWiki:Common.js/edit.js must be loaded both when the action is "edit" and "submit" (for instance the special caracters are loaded whenever during editing); but forcePreview must be loaded only the first time an editpage is loaded, else anonymous visitors couldn't never edit, isn't it? ~ Seb35 [^_^] 5 octobre 2009 à 19:40 (CEST)

Hello!

I was wondering if is it possible to check the value of wgAction only one time. Currently this is checked twice, at MediaWiki:Common.js:

if( wgAction == 'edit' || wgAction == 'submit' ) {
  importScript( 'MediaWiki:Common.js/edit.js' );
}

and at MediaWiki:Common.js/edit.js:

  if (wgUserName != null || wgAction != "edit") return;

But looking at the code now it seems not possible to do this simplification... What do you think? Heldergeovane (d) 17 octobre 2009 à 21:33 (CEST)

Barre Monobook

Trois choses :

• Ce serait bien de préciser qu'il ne s'agit pas d'une barre pour Monobook mais seulement d'une barre dans le style Monobook qu'il y avait auparavant.
• La barre "Vector" par défaut est personnalisable. Du coup, quel est l'intérêt de cette barre "Monobook" ?
• obtenir('MonobookToolbar') -> où est la source ?

— Dakdada (discuter) 3 mai 2013 à 14:02 (CEST)

• En effet.
• Elle est préférée par pas mal d'utilisateurs, et sa création par Mediawiki semble cassée, d'où la volonté de la créer d'ici à notre convenance. Cette barre "monobook style" aussi également personnalisable, de façon probablement plus simple, d'ailleurs.
• Mediawiki:Gadget-MonobookToolbar.js

⇨ D^r Brains ^{∞ Consultation ∞} 3 mai 2013 à 15:39 (CEST)

Merci. Sur le deuxième point, MonobookToolbar est un gadget donc, mais alors pourquoi le charger ici, pour tout le monde ? Il serait plus logique d'activer le gadget par défaut (désactivable si on veut). NB : Y a-t-il un lien qui discute de ce qui « semble cassé » ? — Dakdada (discuter) 3 mai 2013 à 16:03 (CEST)

Maybe MW:Talk:MediaWiki 1.22/wmf3. JackPotte ($♠) 4 mai 2013 à 13:08 (CEST)

@Darkdadaah : c'est ce qui a été fait, finalement. CF les bistros d'hier et d'aujourd'hui.

⇨ D^r Brains ^{∞ Consultation ∞} 4 mai 2013 à 13:57 (CEST)

Explication code dans initializeCharSubset()

Après avoir remarqué cette modif et la suivante, j'ai retrouvé la discussion en rapport : DIMS MediaWiki:Common.js/edit.js. od†n ↗^blah 31 décembre 2017 à 14:50 (CET)

Je viens de modifier le code de sorte à remplacer le javascript inline (href="javascript:...") par un event handler onclick, ce qui permet de résoudre tous ces problèmes d'escaping et d'injection javascript. od†n ↗^blah 31 décembre 2017 à 16:34 (CET)