)]}'
{"horizon/static/horizon/js/horizon.tabs.js":[{"author":{"_account_id":11778,"name":"Diana Whitten","email":"hurgleburgler@gmail.com","username":"hurgleburgler"},"change_message_id":"60459123f86ddec97e929a4e911c51c381a940c6","unresolved":false,"context_lines":[{"line_number":50,"context_line":"  $document.on(\"show.bs.tab\", \".ajax-tabs a[data-loaded\u003d\u0027false\u0027]\", horizon.tabs.load_tab);"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"  $document.on(\"shown.bs.tab\", \".nav-tabs a[data-toggle\u003d\u0027tab\u0027]\", function (evt) {"},{"line_number":53,"context_line":"    var $tab \u003d $(evt.target), $content \u003d $($(evt.target).attr(\u0027data-target\u0027));"},{"line_number":54,"context_line":"    $content.find(\"table.datatable\").each(function () {"},{"line_number":55,"context_line":"      horizon.datatables.update_footer_count($(this));"},{"line_number":56,"context_line":"    });"}],"source_content_type":"text/javascript","patch_set":1,"id":"da6ed579_1a1309ba","line":53,"updated":"2016-01-12 23:29:26.000000000","message":"In general, always use .prop instead of .attr\n\nAlso, there\u0027s no reason to do two jQuery constructors here.  Use the cached $tab.data(\u0027data-target\u0027)...","commit_id":"ca5faf702d75ed38316bbb4994750da73d4734df"},{"author":{"_account_id":17013,"name":"Rajat Vig","email":"rajatv@thoughtworks.com","username":"rajatvig"},"change_message_id":"14e8e7bed0b4a82808a342a3d46b075e55a95c48","unresolved":false,"context_lines":[{"line_number":50,"context_line":"  $document.on(\"show.bs.tab\", \".ajax-tabs a[data-loaded\u003d\u0027false\u0027]\", horizon.tabs.load_tab);"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"  $document.on(\"shown.bs.tab\", \".nav-tabs a[data-toggle\u003d\u0027tab\u0027]\", function (evt) {"},{"line_number":53,"context_line":"    var $tab \u003d $(evt.target), $content \u003d $($(evt.target).attr(\u0027data-target\u0027));"},{"line_number":54,"context_line":"    $content.find(\"table.datatable\").each(function () {"},{"line_number":55,"context_line":"      horizon.datatables.update_footer_count($(this));"},{"line_number":56,"context_line":"    });"}],"source_content_type":"text/javascript","patch_set":1,"id":"da6ed579_e56e96c8","line":53,"in_reply_to":"da6ed579_1a1309ba","updated":"2016-01-12 23:44:14.000000000","message":"This code was as is. I just moved the declaration up.\nI could change to what you suggest if it helps the +1 but that isn\u0027t the intent of this change.","commit_id":"ca5faf702d75ed38316bbb4994750da73d4734df"}]}