Your code did not work, since anchor has no children / children when using the operator $('.list-group a').find('.active') , so you get undefined , here you can use .filter() instead of .find()
var itemtype = $('.list-group a').filter('.active').data('itemtype');
How you added the active class to the previous statement. Direct use of $(e.target).data('itemtype'); ,
According to the HTML, you can also use this instead of e.target .
jQuery(document).ready(function() { $('.list-group-item').on('click', function(e) {
<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script> <div class="list-group"> <a href="#" class="list-group-item" data-itemtype="1">Cras justo odio</a> <a href="#" class="list-group-item" data-itemtype="0">Dapibus ac facilisis in</a> <a href="#" class="list-group-item" data-itemtype="3">Morbi leo risus</a> <a href="#" class="list-group-item" data-itemtype="4">Porta ac consectetur ac</a> <a href="#" class="list-group-item" data-itemtype="2">Vestibulum at eros</a> </div>
source share