rrit
Fevereiro 7, 2022, 8:33pm
14
O feature flag foi implementado.
Eu não sou um desenvolvedor Ruby - nisso eu definitivamente preciso de ajuda.
Talvez enviar meu POC para um novo branch no repositório discourse/discourse, antes de fazer um PR no main?
Este é o meu PR sobre este recurso:
main ← rr-it:dev/javascript-defer
opened 03:49AM - 08 Feb 22 UTC
Implement experimental feature flags for "static topic content" and " javascript… defer"
### static topic content
`SiteSetting.enable_experimental_static_topic_content`
Can be set via settings dashboard.
Show static content in topic view for a faster Largest Contentful Paint (LCP).
Warning: visitors might see disconcerting jumping of content!"
### javascript defer
`SiteSetting.enable_experimental_javascript_defer`
Can be set via settings dashboard.
Defer loading of JavaScripts to show static content even faster in combination with 'enable_experimental_javascript_defer'. Warning: the discourse frontend, themes, components and/or plugins might fail!
---
The vendor-javascript and all preceding javascripts are not deferred right now.
@see: https://github.com/rr-it/discourse/commit/49405c353a31180933aed9dca7697dce05227707
Ideas on how to solve this are very welcome.
---
For more information see also:
https://meta.discourse.org/t/defer-javascript-and-show-interim-content-on-initial-page-load/216458
@david Você pode me dar uma mãozinha para desenvolver testes Rspec para essas alterações:
app/helpers/application_helper.rb : spec/helpers/application_helper_spec.rb
Eu não vejo testes unitários viáveis aqui. Parece testável apenas por testes de integração.app/models/theme.rb app/models/theme_field.rb
Eu tive que desabilitar o defer tag para o QUnit Test Runner: app/views/qunit/index.html.erb "javascript defer" = false. E agora os testes rodam também com "javascript defer" = true.
2 curtidas