$(this).click(function(){ // и по нажатию на неё
зачем еще раз событие клика навешивать? Оставьте только
thisTabItemsWrapper.css('max-height', defaultMaxHeight); //меняем высоту списка табов на дефолт
$(this).html('показать ещё'); // меняем текст кнопки на стандартный
Все равно если условие не выполнится то и этот кусок кода не сработает
upd:: дошло зачем
замените это
$(this).html('показано всё (скрыть)'); //меняем текст кнопки
if((this).html() == 'показано всё (скрыть)'){
thisTabItemsWrapper.css('max-height', defaultMaxHeight); //меняем высоту списка табов на дефолт
$(this).html('показать ещё'); // меняем текст кнопки на стандартный
}
и пишите
if($(this).html() == 'показано всё (скрыть)'){
thisTabItemsWrapper.css('max-height', defaultMaxHeight); //меняем высоту списка табов на дефолт
$(this).html('показать ещё'); // меняем текст кнопки на стандартный
}
else $(this).html('показано всё (скрыть)'); //меняем текст кнопки
конечно $(this).html() == 'показано всё (скрыть)' не есть хорошо, лучше загонять значение в какой нибудь data-button-state = 'default | hide'