angular-ui/ui-grid

Remove 'replace' from directives

Open

#4024 aperta il 21 lug 2015

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (2496 fork)batch import
good first issueneeds: decisiontype: enhancement

Metriche repository

Star
 (5395 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Two major reasons:

  • It is deprecated and will be removed in Angular 2.0 https://docs.angularjs.org/api/ng/service/$compile
  • It creates more problems than it is worth.
    • For example: If you try to create a stacked directive and start adding classes to an element they end up getting removed because the dom element gets removed.

Guida contributor

Direzione di ricerca
Identifica tutte le direttive che hanno una proprietà 'replace' ed elimina quella proprietà. Quindi esegui la suite di test per assicurarti che nulla si rompa.
Tech stack
javascriptangular
Dominio
frontend
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
JavaScriptGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75