Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

How to prevent ng-click's triggering twice on label tag?

Angular's ng-click gets triggered twice when I click label which has input inside in it. I've tried $event.stopPropagation(); but didn't work. How do I solve this?

I've checked this question also: Angular.js ng-click events on labels are firing twice

<div class="list-group-item" ng-repeat="item in model.data">
  <form role="form" name="selectForm" novalidate>
    <label ng-click="$event.stopPropagation(); updateSelected();">
      <input type="checkbox" ng-model="chechkedSkins[item.id]" />
      <span>{{item.name}}</span>
    </label>
  </form>
</div>
like image 265
Barlas Apaydin Avatar asked Sep 01 '15 11:09

Barlas Apaydin


2 Answers

Use ng-change="updateSelected"

Use this only on the input since the change is triggered even if you click the label.

like image 93
Adrian Florescu Avatar answered Sep 22 '22 01:09

Adrian Florescu


Well thats because label is the parent or container for the checkbox, so the click handler is attached to the complete container in your case, thereby whenever either label or checkbox is clicked, event is triggered.


Whats wrong with your approach:

  • Firstly never insert the input tags inside the label, thats not a good way to construct markup in html.In Angular.js this behaviour causes the click event to be triggered for both the tags. so to add a binding between input tag & label use the for attribute of label.
  • Using $event.stopPropagation() inside label actually stops all events from propagating/boiling to the top of the DOM from the label. this will not serve any purpose because the event would still propagate to the input with in the label.

I hope you can visualise what i'm saying.

What I have done:

  • Use for attribute to bind the input to the label & add a click event to prevent the default functionality.
  • Add the click handler to the respective input tag & not the label

    <label for="username" ng-click="$event.preventDefault();">Click me</label> <input type="text" id="username" ng-click="updateSelected();">

Live Demo @ JSFiddle

This way you don't have to worry about any conflicts in event handling,also its neat way to maintain the HTML code :)

like image 29
dreamweiver Avatar answered Sep 24 '22 01:09

dreamweiver