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

thulioph/reorder-imports #300

Open
wants to merge 11 commits into
base: develop
Choose a base branch
from

Conversation

thulioph
Copy link

@thulioph thulioph commented Jun 1, 2020

Atualizações

As atualizações deste pull request são relacionadas a otimização no carregamento da página, e resolve algumas issues do repositório:

Foram feitas mudanças na ordenação dos scripts, em como alguns assets são "baixados", e algumas pequenas adições (eventListeners, if/else) para as mudanças surtirem efeito.

  • Removi o CDN de 3 bibliotecas (leaflet.js, jquery.dataTables.min.js e Chart.min.js) e agora temos elas de maneira local.

  • Removi todas as execuções de scripts de dentro dos arquivos .html e todos se encontram no base.html.

  • Algumas pequenas execuções foram concentradas no arquivo application.js.

  • Para os arquivos relacionados a app do COVID19, a solução inicial foi verificar qual URL o usuário se encontra para ai executar os scripts.

  • Utilizei o preload para o font-awesome visando evitar o bloqueio na renderização da página.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant