MediaWiki Discussão:Gadget-tutoring-enrollment.js

formatStr

@Diego Queiroz: sugiro que tente utilizar mw.Message em vez de criar uma formatStr para uma função similar. Helder 00h31min de 9 de novembro de 2014 (UTC)

Muito bom! Eu tinha visto esse mw.Message nos FastButtons mas não tinha entendido direito como funcionava. Assim que tiver tempo eu ajusto o código. --Diego Queiroz (discussão) 01h23min de 9 de novembro de 2014 (UTC)

Atalhos

Seria melhor utilizar o módulo jquery.accessKeyLabel do MediaWiki para mostrar isso corretamente dependendo da plataforma. Helder 00h36min de 9 de novembro de 2014 (UTC)

Anotado! Quando coloquei imaginei isso, mas pensei que a própria Wikipédia desprezava isso. Depois vou arrumar também. --Diego Queiroz (discussão) 01h23min de 9 de novembro de 2014 (UTC)

const

Não acho uma boa ideia o uso da declaração const, pelo fato de que ela só é suportada no IE a partir da versão 11.
Além disso, é sempre bom que constantes tenham o seu nome todo em CAIXA ALTA, afim de facilitar a diferenciação delas com as variáveis. !Silent (discussão) 01h41min de 9 de novembro de 2014 (UTC)

Essa do const eu não sabia, obrigado pelo aviso, acabei de remover. Quanto à questão estética, vou deixar pra outra hora: vou priorizar a mudança para o mw.Message que vai acabar com essas constantes. --Diego Queiroz (discussão) 03h50min de 9 de novembro de 2014 (UTC)

Na verdade, era pra substituir os const por var =) Porque se não seu código fica cheio de variáveis globais, e percebe-se que não é essa a sua intenção, pois no código existe uma IIFE. Mas como você já vai substituir por mw.message, não tem problema. !Silent (discussão) 11h57min de 9 de novembro de 2014 (UTC)