Skip to content
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

Post Status - Otimizações no JavaScript #425

Merged
merged 3 commits into from
Oct 4, 2016

Conversation

HenriqueSilverio
Copy link

Olá pessoal, segue uma breve descrição das mudanças que fiz nesse arquivo.

1: 932a25e Mudando o evento load para o ready (usando sintaxe $( handler )), visto que não precisamos esperar que todos os assets sejam carregados para executar essa função.

2: 69d382f Um cache simples para evitar repetições de consulta ao DOM.

Referências:

@HenriqueSilverio HenriqueSilverio changed the title Post Status - Pequenas melhorias no JavaScript Post Status - Otimizações no JavaScript Oct 4, 2016
@renatonascalves
Copy link

Boas alterações!

@claudiosanches
Copy link
Member

@HenriqueSilverio $( 'body' ) demora mais que $( document.body ).

@renatonascalves
Copy link

@claudiosmweb Sério?! o.O Essa é nova pra mim! :/

@claudiosanches claudiosanches merged commit 2594665 into wpbrasil:master Oct 4, 2016
@claudiosanches
Copy link
Member

@renatonascalves sim, aprendi isso porque me fizeram trocar todos desses no WooCommerce xD

@HenriqueSilverio
Copy link
Author

HenriqueSilverio commented Oct 4, 2016

@claudiosmweb Realmente não tinha parado para pensar nisso, mas faz sentido, pois usando o seletor $( document.body ) o jQuery não vai precisar processar a string e tal para pegar o elemento por baixo dos panos né. Valeu!

@renatonascalves
Copy link

@claudiosmweb Interessante! Acho que vou trocar por aqui também! =P

@claudiosanches
Copy link
Member

@HenriqueSilverio valeu velho :)

@adammacias adammacias added this to the v.2.3.0 milestone Oct 7, 2016
@HenriqueSilverio HenriqueSilverio deleted the post-status branch October 9, 2016 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants