172

I'm relatively new to AngularJS and suspect I'm not grasping a concept. I'm also using Twitter Bootstrap and I've got jQuery loaded.

Workflow: User clicks a link from a list, "master" section is updated and link user clicked on gains active class.

Basic HTML Markup:

<ul class="list-holder" ng-controller="adminController">
   <li><a ng-click="setMaster('client')">Clients</li>
   <li><a ng-click="setMaster('employees')">Employees</li>
   <li><a ng-click="setMaster('etc')>Etc...</li>
</ul>

Doing this in jQuery:

jQuery(".list-holder").on('click', 'a', function(event){
    event.preventDefault();
jQuery(".list-holder li").removeClass('active');
jQuery(this).parent('li').addClass('active');
});

But I can't figure out how to integrate Angular and jQuery to get this done, because I'm using Angular to fetch the master list (in JSON form) from the server and update a list on the page.

How do I integrate this? I can't seem to find the element I've clicked on once I'm inside the angular controller function

Controller:

function adminController($scope)
    {    
        $scope.setMaster = function(obj)
        {
            // How do I get clicked element's parent li?
            console.log(obj);
        }
    }

1 Answer 1

283

While AngularJS allows you to get a hand on a click event (and thus a target of it) with the following syntax (note the $event argument to the setMaster function; documentation here: http://docs.angularjs.org/api/ng.directive:ngClick):

function AdminController($scope) {    
  $scope.setMaster = function(obj, $event){
    console.log($event.target);
  }
}

this is not very angular-way of solving this problem. With AngularJS the focus is on the model manipulation. One would mutate a model and let AngularJS figure out rendering.

The AngularJS-way of solving this problem (without using jQuery and without the need to pass the $event argument) would be:

<div ng-controller="AdminController">
    <ul class="list-holder">
        <li ng-repeat="section in sections" ng-class="{active : isSelected(section)}">
            <a ng-click="setMaster(section)">{{section.name}}</a>
        </li>
    </ul>
    <hr>
    {{selected | json}}
</div>

where methods in the controller would look like this:

$scope.setMaster = function(section) {
    $scope.selected = section;
}

$scope.isSelected = function(section) {
    return $scope.selected === section;
}

Here is the complete jsFiddle: http://jsfiddle.net/pkozlowski_opensource/WXJ3p/15/

12
  • 198
    FYI: $event doesn't work unless it's passed in the markup: ng-click="setMaster(section, $event)" Just a heads up.
    – Ben Lesh
    Oct 21, 2012 at 23:06
  • 4
    Not really, it has to be passed in to the function. In reality, though, it's probably not the best idea to reference dom-specific stuff like this in your controller. Generally when $event is used, it's in the context of stopping propagation or the like: <a ng-click="doSomething(); $event.stopPropagation()">Click Just Me</a>
    – Ben Lesh
    Mar 10, 2014 at 16:51
  • 9
    I had a problem using the $event.target because I had a icon inside my button. So, sometimes the target result is the button and sometimes is the icon (depending where I clicked). I used $event.currentTarget instead of target and it worked like a charm.
    – lao
    May 10, 2014 at 14:29
  • 4
    Though using $event.target like this may not be the "angular way" I feel like having a large ng-repeat list with every item needing a listener to evaluate a change is not efficient? Clicking one element would mean every item in the entire list must be re-evaluated right? -- why not just target that one item with $event.target to toggle a CSS class for example. What do you think? I'm working on a Phonegap App and need to squeeze every performance tweak I can. Jun 19, 2015 at 3:47
  • 13
    I would also like to recommend using $event.currentTarget instead of $event.target. If the element with ng-click has child elements, if the child element is clicked $event.target becomes the child element. $event.currentTarget will always target the element with the designated ng-click. Jun 19, 2015 at 20:36

Not the answer you're looking for? Browse other questions tagged or ask your own question.