nytimes/library

Audit script load locations, inline code

Open

#53 aperta il 11 ago 2019

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)JavaScript (148 fork)auto 404
enhancementgood first issue

Metriche repository

Star
 (1180 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

As Isaac said in #51, we could likely move many (if not all) of our script tags out of the document <head>.

It looks like all the other Library scripts are in the head (and we just haven't audited this in a while). [...] I think it would be worthwhile for us to be consistent about where scripts are included. [...] What do you think about moving the other includes so they can all be together? My hunch is that this wouldn't break anything but would be curious to see what you find.

Originally posted by @isaacwhite in https://github.com/nytimes/library/pull/51

Additionally, as seen in #283, our inline client-side code could probably use some cleanup as well.

Guida contributor

Direzione di ricerca
Controlla le attuali posizioni di caricamento degli script nei file HTML, identifica quali possono essere spostati fuori da <head> e proponi un posizionamento coerente. Rivedi anche il codice inline per opportunità di pulizia.
Tech stack
javascript
Dominio
frontenddocumentation
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitHTMLJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75