角ng-click事件

Question

I have these functions:

$scope.addProduct = function($event, obj) {

    var myEl = angular.element($event.target);

    myEl.parent().html('<button class="btn btn-danger btn-xs" ng-click="removeProduct('+ obj.id +')">Remove from Widget</button>');

};

$scope.removeProduct = function(id) {
    console.log('Remove ->' + id);

};

When I click on "addProduct" it work but when I click on "removeProduct" does not work. Looks like it doesn't listen the click on the new button that I added. How can I solve it?

Answer 1

In order for ngClick to take effect, you need to first compile your HTML and link it to the $scope :

.controller('someCtrl', function ($compile, $scope) {
    ...

    $scope.addProduct = function($event, obj) {
        var myEl = angular.element($event.target);
        var myBtn = $compile('<button class="btn btn-danger btn-xs" ng-click="removeProduct('+ obj.id +')">Remove from Widget</button>')($scope);
        myEl.after(myBtn);
    };

    $scope.removeProduct = function(id) {
        console.log('Remove ->' + id);
    };
});

---

UPDATE:

Kos Prov's comment is very true - sometimes you think about solving the problem, but you don't think the question was wrong in the first place.

So, of course manipulating the DOM should be a directive's responsibility, so I created a directive to do what you want (according to your question):

app.directive('removableProduct', function ($compile) { var btnTmpl = '' + 'Remove from Widget' + '';

    return {
        restrict: 'A',
        scope: {item: '='},
        template: '<div ng-click="addProduct($event)">{{item.description}}</div>',
        controller: function ($scope) {
            $scope.addProduct = function (evt) {
                var myEl = angular.element(evt.target);
                var myBtn = $compile(btnTmpl)($scope);
                myEl.after(myBtn);
            };
            $scope.removeProduct = function(id) {
                console.log('Remove -> ' + id);
            };
        }
    };
});

---

See, also, this short demo .

Answer 2

You have a problem which should not exist if you do it in Angular way.

For every product try to hold a boolean value like 'productAdded' in your $scope model.

Then you can do in example:

<div ng-repeat="product in products">
    <button ng-show="product.productAdded" class="btn btn-danger btn-xs" ng-click="addProduct(product)">Add to Widget</button>
    <button ng-show="!product.productAdded" class="btn btn-danger btn-xs" ng-click="removeProduct(product.id)">Remove from Widget</button>
</div>

Hope this clarifies a bit for you!

Answer 3

If you want to add a remove button after a product is added, you should add the remove button within the ng-repeat itself:

JS

<div ng-app="app" ng-controller="ctrl">
   <ul>
      <li ng-repeat="product in products">
           {{product.name }}<button ng-click="deleteProduct(product)">Delete</button>
      </li>
   </ul>
</div>

Controller

var app = angular.module('app', []);
app.controller('ctrl', function($scope) {
    $scope.products = [{ name:'pencil' }, { name: 'crayon' }, { name: 'pen' }];
    $scope.deleteProduct = function(product) {
        var index = $scope.products.indexOf(product);
        if (index >=0) {
            // remove the product
            $scope.products.splice(index, 1);
        }
    }
});