Упрощение кода

@Icons · Регистрация: 04.04.2015

Студворк — интернет-сервис помощи студентам

Всем привет!!
Попалась на вёрстку менюшка с противными разделителями...ну совсем не стандартными. На макете, вроде, всё красиво, но на деле....У LI бэкграундом сделана полоска-картинка...При ховере и активности разделители с обоих сторон должны пропадать...
Короче пришлось приложить свои "знания" jQuery. Всё получилось, всё работает как надо, но уж больно позорный код получился)))
Может кто-нибудь знает как его можно упростить? Помогите если не трудно..

HTML5
1
2
3
4
5
6
<ul class="top-menu">
    <li><a href="">наши бренды</a></li>
    <li  class="selected"><a href="">сеть отделов</a></li>
    <li><a href="">дисконты</a></li>
    <li><a href="">новости</a></li>
</ul>

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
$(document).ready(function() {
$( "ul.top-menu li" ).hover(
  function() {
    $( this ).prev().css("background", "none");
    $( this ).css("background", "none");
  }, function() {
    $( this ).prev().css("background", "url('images/razdelit.png')center right no-repeat");
    $( this ).css("background", "url('images/razdelit.png')center right no-repeat");
    $("ul.top-menu li:last-child").css("background", "none");
    $("ul.top-menu li.selected").css("background", "none");
    $("ul.top-menu li.selected").prev().css("background", "none");
  }
);
 
     if($("ul.top-menu li").hasClass("selected"))
        $('ul.top-menu li.selected').prev().css("background", "none");
        $('ul.top-menu li.selected').css("background", "none"); 
});

CSS
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
ul.top-menu{
    margin: 0;
    padding: 0;
    list-style: none;
}
ul.top-menu li{
    position: relative;
    display: inline-block;
    position: relative;
    background: url('../images/razdelit.png')center right no-repeat;
}
 
ul.top-menu li:last-child, ul.top-menu li:last-child:hover{
    background-position: -9999px 0px;
        
}
 
ul.top-menu li a{
    text-decoration: none;
    text-transform: uppercase;
    color: #81714b;
    font: 16px  'Basil_Regular';
    padding: 10px 23px;
    border:1px solid rgba(255,255,255,0);
}
ul.top-menu li a:hover, ul.top-menu li.selected a{
    border:1px solid #385d9d;
    color: #335889;
}

@o90990 · 22.11.2015, 10:53

весь js можно перенести на стили, не понимаю зачем тут он вобще нужен

@Icons · 22.11.2015, 12:08 **[ТС]**

Нет, в стилях не получится...тут нужно при наведении отменять стиль у предыдущего элемента в списке...В jQuery есть хорошая фича .prev(), а вот в CSS такого нет, а хотелось бы...
А так то вообще сильно позорный код или пойдёт?

@Lazy_Den · 22.11.2015, 12:39

Сообщение от Icons

в стилях не получится

Даже не особо заморачиваясь, можно получить такой результат. Основные изменения:

CSS
1
2
3
4
5
6
7
8
9
10
11
ul.top-menu li:not(:first-child)::before {
    display: inline-block;
    content: url('../images/razdelit.png');
}
ul.top-menu li:hover::before,
ul.top-menu li:hover + li::before,
ul.top-menu li.selected  + li::before,
ul.top-menu li.selected::before{
    content: '';
    width: 1px; /* ширина изображения razdelit.png */
}

При желании, можно и оптимизировать.

@Icons · 22.11.2015, 23:19 **[ТС]**

Хм...точно...
Что-то не додумался....Спасибо.

Новые блоги и статьи Все статьи Все блоги /
Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .	Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .	Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .
SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip Сканируйте QR-код на мобильном и вы увидите, что появится джойстик для управления главным героем. . . .	Реалии Hrethgir 01.03.2026 Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .	Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .

@o90990 8 / 8 / 7 Регистрация: 19.02.2012 Сообщений: 76
	22.11.2015, 10:53
	весь js можно перенести на стили, не понимаю зачем тут он вобще нужен 0

@Icons 1 / 1 / 2 Регистрация: 04.04.2015 Сообщений: 66
	22.11.2015, 12:08 [ТС]
	Нет, в стилях не получится...тут нужно при наведении отменять стиль у предыдущего элемента в списке...В jQuery есть хорошая фича .prev(), а вот в CSS такого нет, а хотелось бы... А так то вообще сильно позорный код или пойдёт? 0

@Icons 1 / 1 / 2 Регистрация: 04.04.2015 Сообщений: 66
	22.11.2015, 23:19 [ТС]
	Хм...точно... Что-то не додумался....Спасибо. 0

Опции темы