angular-ui/ui-grid

Remove 'replace' from directives

Open

#4.024 aberto em 21 de jul. de 2015

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)JavaScript (2.496 forks)batch import
good first issueneeds: decisiontype: enhancement

Métricas do repositório

Stars
 (5.395 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
Identifique todas as diretivas que possuem uma propriedade 'replace' e exclua essa propriedade. Em seguida, execute o conjunto de testes para garantir que nada quebre.
Pilha de tecnologia
javascriptangular
Domain
frontend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScriptGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75