MediaWiki Discussão:Gadget-tutoring-enrollment.js
Adicionar tópicoAspeto
Último comentário: 9 de novembro de 2014 de !Silent no tópico const
formatStr
[editar código-fonte]@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
[editar código-fonte]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
[editar código-fonte]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
porvar
=) 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 pormw.message
, não tem problema. !Silent (discussão) 11h57min de 9 de novembro de 2014 (UTC)
- Na verdade, era pra substituir os