Используйте больше CSS в таблице стилей и меньше встроенных.Тогда вы можете просто переключать класс и стоять и смотреть ... классный .Еще одна вещь: вы можете связать только один click
прослушиватель событий, и в обработчике событий проверьте, нужно ли вам открывать или закрывать меню.
CSS
.userNavBar {
background-color: #333;
}
.userNavBar.active {
background-color: #444;
}
.upperBar {
border-top-color: #333;
}
.upperBar.active {
border-top-color: #ff556f;
}
JavaScript
$(function() {
function toggleMenu(show) {
$('.dropdownInfo').toggle(show);
$('.userNavBar, .upperBar').toggleClass('active', show);
};
$('body').on('click', function(e) {
var show = $(e.target).hasClass('userNavBar');
if (show) e.preventDefault(); // not sure this is even necessary anymore
toggleMenu(show);
});
});
Также, пожалуйста, не смешивайте одинарные и двойные кавычки.Выберите один и будьте последовательны.