let's try category class again

This commit is contained in:
Régis Hanol 2015-01-27 23:44:47 +01:00
parent a82c50d584
commit b003c2b4f3
2 changed files with 19 additions and 23 deletions

View file

@ -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')
};

View file

@ -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");