-
Notifications
You must be signed in to change notification settings - Fork 195
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Novas classes de opções do tema e metabox #141
Comments
Acho válido demais. Realmente ficou bem mais fácil entender. |
Eu curti! Ficou mais fácil separar as coisas.. dá pra deixar os métodos bem menores agora sem precisar dividir os arrays e dar merge depois xD Só uma dúvida: vai mudar o core pra funcionar assim, ou só vai criar métodos pra consumir os que já existem hoje? Só pra eu saber, pois estou trabalhando (a passos curtos xD) no esquema do checkbox pra ficar habilitado por padrão e no repeater :) |
Sorry, não tinha visto o commit xD Bom, pelo que vi, vai ficar bem mais fácil pra implementar o repeater agora o/ Vai ficar as duas versões ativas (anterior e essa) ou somente essa? Pergunto mais pela questão da atualização. Se a galera estiver usando a versão anterior e migrar pra essa vai quebrar né? Daria talvez pra fazer uma classe "Odin_Migrate" pra quem quiser usar a versão nova, mas já tiver implementado com a versão antiga xD |
A ideia é praticamente zerar mesmo e fazer desta forma bem melhor. |
Muito bom! Vai diminuir bastante a repetição de código pra criação dos custom fields =D Nessa versão você também pretende já remover o Grunt pra usar como submódulo? Vou deixar a doc do Gulp certinha já pra mostrar como usar ele dessa forma. Depois dá pra fazer um só pro Grunt também :) |
Então, não sei ainda, vamos avaliar isso do Grunt. Essas classes ai acho que vai ser rápido, o mais complicado esta pronto já xD |
Massa! Let's go on! o/ |
Além de economizar código, ficou super simples! Aprovado. |
Fechando esta issue aqui, movida para o Odin Toolkit: wpbrasil/odin-toolkit#4 |
Já tem algum tempo que não estou satisfeito com o jeito que criamos as opções do tema, metaboxes e outras coisas.
Hoje funciona muito bem do jeito que esta, entretanto acaba ficando confuso para quem não manja muito ficar criando aquela infinidade de arrays que fazemos hoje!
Desta forma pensei em criar isso de uma forma mais simples, acabei de fazer uma primeira vez com as opções do tema apenas (depois preciso abstrair mais para poder usar a mesma classe de campos para os metaboxes também e etc) para ver como que fica e se todo mundo aprova.
As classes estão no branch
new_settings
para quem quiser testar e aqui vai um exemplo de como usar:Acredito que desta forma fica muito mais claro visualizar que tipo de campo você esta criando e dentro de qual sessão e em qual aba xD
Aqui como fazemos hoje para vocês comparem:
http://wpod.in/pt-br/docs/criando-opcoes-para-tema/#exemplo
Comentem!
Commit: 9c2248c
The text was updated successfully, but these errors were encountered: