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

@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 **[ТС]**

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

Новые блоги и статьи Все статьи Все блоги /
Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .	SDL3 для Web (WebAssembly): Сборка библиотек SDL3 и Box2D из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия SDL 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual. . .	SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.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