Возможно ли переделать условие для скроллинга?

Question

[River0051]

Всем привет, возможно ли переделать условие таким образом, чтобы скрипт работал только для блока с id tabs ?

Что делает скрипт: при переходе по ссылке с хешем, прокручивает страницу до нужного id
Что хотелось бы: чтобы прокручивало только тогда, если имя хеша совпадает с id внутри блока #tabs и не работало если внутри блока #tabs нет такого id который совпадает с хешем в ссылке

$(document).ready(function(){
var loc = window.location.hash.replace("#","");
if (loc == "") 
return false;
var destination = $("#"+loc).offset().top;
$('html, body').animate({ scrollTop: destination-80}, 1500 );
});

Answer 1

[Stalker_RED]

Проверяем есть ли блок с нужным id внутри #tabs:

if (location.hash && $('#tabs ' + location.hash).length ) {
  // ...
}

Демо: https://jsfiddle.net/eqkb62Lv/1/

Comments

[River0051]

Не работает, я не туда вставил условие ?
$(document).ready(function(){
var loc = window.location.hash.replace("#","");
if (loc == "")
return false;
if (location.hash && $('#tabs ' + location.hash).length ) {
var destination = $("#"+loc).offset().top;
$('html, body').animate({ scrollTop: destination-80}, 1500 );
}});

[Stalker_RED]

Станислав: Нет, работает https://jsfiddle.net/eqkb62Lv/

[Stalker_RED]

Станислав: И этот ваш var loc и replace - не нужны.
https://jsfiddle.net/eqkb62Lv/1/

[River0051]

Спасибо, все работает, if loc дописывал чтобы браузер не ругался на то, что хеша нет

[Stalker_RED]

Станислав: Дык if (location.hash && ... не просто так же.

[River0051]

я просто объяснил зачем это было там, не знаю зачем объяснял, показалось, что это нужно сделать.

[Stalker_RED]

Станислав: Да я понял что это проверка такая, но во-первых получилось дублирование.
Во вторых, вы зачем-то отрезали от хеша '#' а потом опять добавляли $("#"+loc)
Суета :)

[River0051]

Понятно, спасибо