Remove 'replace' from directives · angular-ui/ui-grid#4024

2015-07-21T20:31:41.000Z

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.

(0 comments) (0 reactions) (0 assignees)JavaScript (5,395 stars) (2,496 forks)batch import

good first issueneeds: decisiontype: enhancement

説明

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.

コントリビューターガイド

技術スタック: javascriptangular
領域: frontend
Issue 種別: refactor
難度: 2
推定時間: 1-3 hours
活動状況: stale
明確さ: clear
前提条件: AngularJSDirectives
初心者向け度: 50
調査方針: This issue involves removing the deprecated 'replace' property from all AngularJS directive definitions in the ui grid codebase. Look for occurrences of 'replace: true' or 'replace: false' in directive definition objects. Ensure tests are updated accordingly. The issue is from 2015, so verify if the codebase still uses AngularJS version that supports it. Refer to the AngularJS documentation on $compile for context.