diff --git a/app/assets/javascripts/discourse/mixins/add-category-class.js.es6 b/app/assets/javascripts/discourse/mixins/add-category-class.js.es6 index 35621ae5bce..209b105f36a 100644 --- a/app/assets/javascripts/discourse/mixins/add-category-class.js.es6 +++ b/app/assets/javascripts/discourse/mixins/add-category-class.js.es6 @@ -2,25 +2,22 @@ // add it to the body as the view is entered / left / model is changed. // This is used for keeping the `body` style in sync for the background image. export default { - // Sam: Something about this code is messing with the "docked" class on body - // it looks good, but something weird is going on. - // - // _enterView: function() { this.get('categorySlug'); }.on('init'), - // - // _removeClasses: function() { - // $('body').removeClass(function(idx, css) { - // return (css.match(/\bcategory-[^\b]+/g) || []).join(' '); - // }); - // }, - // - // _categoryChanged: function() { - // var categorySlug = this.get('categorySlug'); - // this._removeClasses(); - // - // if (categorySlug) { - // $('body').addClass('category-' + categorySlug); - // } - // }.observes('categorySlug'), - // - // _leaveView: function() { this._removeClasses(); }.on('willDestroyElement') + _enterView: function() { this.get('categorySlug'); }.on('init'), + + _removeClasses: function() { + $('body').removeClass(function(idx, css) { + return (css.match(/\bcategory-\S+/g) || []).join(' '); + }); + }, + + _categoryChanged: function() { + var categorySlug = this.get('categorySlug'); + this._removeClasses(); + + if (categorySlug) { + $('body').addClass('category-' + categorySlug); + } + }.observes('categorySlug'), + + _leaveView: function() { this._removeClasses(); }.on('willDestroyElement') }; diff --git a/test/javascripts/integration/topic-discovery-test.js.es6 b/test/javascripts/integration/topic-discovery-test.js.es6 index 33425a7ac23..ea4dbac42c2 100644 --- a/test/javascripts/integration/topic-discovery-test.js.es6 +++ b/test/javascripts/integration/topic-discovery-test.js.es6 @@ -11,8 +11,7 @@ test("Visit Discovery Pages", function() { andThen(function() { ok(exists(".topic-list"), "The list of topics was rendered"); ok(exists('.topic-list .topic-list-item'), "has topics"); - // TODO enable test once fixed - // ok($('body.category-bug').length, "has a custom css class for the category id on the body"); + ok($('body.category-bug').length, "has a custom css class for the category id on the body"); }); visit("/categories");