Javascript-форум (https://javascript.ru/forum/)
-   jQuery (https://javascript.ru/forum/jquery/)
-   -   Конфликт скриптов (https://javascript.ru/forum/jquery/52260-konflikt-skriptov.html)

рони 11.12.2014 15:48

http://javascript.ru/forum/misc/3852...tml#post254605

рони 11.12.2014 15:50

Poznakomlus,
а почему document.location.href а не window.location.href

Deff 11.12.2014 15:55

лучше так:
$('a[href*="//'+location.hostname+'"]').addClass('sushi-active');

Vlasenko Fedor 11.12.2014 16:03

рони,
так написал. И даже не скажу про разницу в данном вопросе между location объекта window и document
Deff,
согласен

Deff 11.12.2014 16:07

Нет смысла выделять ссылки на текущую страницу, если они не связаны с разными якорями(их вообще стоит скрыть), отличными от текущего
А при наличии якоря, приведённое выделение не будет работать на все остальные якоря, поэтому см Выше

danik.js 11.12.2014 16:08

В подавляющем большинстве (уверен, и у ТС в том числе) ссылка будет такой:
<a href="/showhtml">Active link</a>

Deff 11.12.2014 16:11

danik.js,
Сейчас все стараются ставить полную ссылку, см статьи на Хабре, в частности в связи с разными уязвимостями
Можно глянуть самые популрные сайты и поисковики, в принципе укорочение кода на один слеш мало повлияет на вероятность ложного срабатывания

danik.js 11.12.2014 16:22

Цитата:

Сообщение от Deff
Сейчас все стараются ставить полную ссылку, см статьи на Хабре, в частности в связи с разными уязвимостями

Кинь ссылку. Что-то не встречал. Даже не знаю как гуглить. Что за уязвимости? Где?

danik.js 11.12.2014 16:26

Кстати файрффокс походу атрибут подменяет абсолютной ссылкой. ие может также делает. хром нет.
Цитата:

Сообщение от Deff
Можно глянуть самые популрные сайты и поисковики

Например тот же Habr?


Часовой пояс GMT +3, время: 11:58.