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

Fix coding standards #405

Merged
merged 1 commit into from
Jun 28, 2016

Conversation

rodrigo-brito
Copy link
Member

E ae pessoal, submeti um tema para o repositório oficial utilizando o Odin. Passaram um pente fino na revisão. Entre os itens que mais pegaram no pé foi os escapes. Aproveitei e já atualizei por aqui.

Outra sugestão que deram foi a utilização dos condicionais do IE através do:

wp_script_add_data( 'html5shiv', 'conditional', 'lt IE 9' );

Uma outra coisa foi o select com tamanho muito grande, ele estava estourando o layout, coloquei um max-width: 100%;. O problema é mostrado na imagem abaixo:

quebra_sidebar

@claudiosanches
Copy link
Member

Puts, pior que quando saiu o WordPress 4.2 eu vi o wp_script_add_data() e pensei em colocar no Odin, mas até agora nada xD
Valeu cara :)

@claudiosanches claudiosanches merged commit 1a95ee5 into wpbrasil:master Jun 28, 2016
@rodrigo-brito
Copy link
Member Author

Eu não tinha visto isso ainda, achei bacana d+.

@adammacias adammacias added this to the v.2.3.0 milestone Jun 28, 2016
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.

3 participants