Опять таки не понятно почему не работает скрипт

@Denim · Регистрация: 05.07.2014

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

Есть простая разметка

HTML5
1
2
3
4
5
<div class="header-wrap">
 <ul class="navigation">
  <li>Привет</li>
 </ul>
</div>

CSS
1
2
3
4
.header-wrap{
height:50px;
background:green;
}

И скрипт

JavaScript
1
2
3
4
5
6
7
8
window.onload = function(){
var headerWrap = document.getElementsByClassName("header-wrap");
var navigation = document.getElementsByClassName("navigation");
if(navigation.onmouseover)
headerWrap.style.height = "100px";
else if(navigation.onmouseout)
headerWrap.style.height = " ";
};

При наведении на блок с классом navigation, блок с классом header-wrap должен меняться в размере.
Что не так, где ошибка?

quwy · 22.08.2014, 19:16

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

Что не так

Все не так. Откуда вы этот бред взяли вообще?

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

где ошибка?

В нем самом

@Denim · 22.08.2014, 19:19 **[ТС]**

quwy, мои знания совсем поверхностные в js.
что то не так в условии?

quwy · 22.08.2014, 19:23

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

мои знания совсем поверхностные в js.

Мои тоже.

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

что то не так в условии?

Я же сказал, что все.

Вы повесили свой код на событие завершения загрузки страницы. Этот код выполнится один раз. Что я вижу в этом коде? В нем вы проверяете наличие обработчиков onmouseover/onmouseout и немедленно делаете манипуляции со стилем. Ни о какой реакции на мышь тут вообще речь не идет.

@nubideus · 22.08.2014, 19:23

getElementsByClassName возвращает HTMLCollection, а это почти массив.
> if(navigation.onmouseover)
события так не работают

JavaScript
1
2
3
4
5
6
7
8
var headerWrap = document.getElementsByClassName("header-wrap")[0]; // << возвращает массив
var navigation = document.getElementsByClassName("navigation")[0]; // тоже
navigation.onmouseover = function(){ // почему ты window.onload правильно написал, а это не?
    headerWrap.style.height = "100px";
};
navigation.onmouseout = function(){
    headerWrap.style.height = ""; //<< нет пробела
};

http://jsfiddle.net/e5q28gL2/

в onload сам оберни

@Padimanskas · 22.08.2014, 19:31

Denim, вам нужно установить обработчик на события omouseover и onmouseout.

JavaScript
1
2
3
4
var handler = function (){
    Do something
}
addEventListener('mouseover', handler, false);

quwy · 22.08.2014, 19:36

Padimanskas, это ж как нужно любить анонимные методы, чтобы использовать их даже там, где сам здравый смысл против.

@nubideus · 22.08.2014, 19:42

quwy, всегда так делаю, систайл.
он разве что ; забыл, глаза резануло

@Denim · 22.08.2014, 19:46 **[ТС]**

nubideus, сделал как посоветовали:

JavaScript
1
2
3
4
5
6
window.onload = function(){
var headerWrap = document.getElementsByClassName("header-wrap")[0];
var navigation = document.getElementsByClassName("navigation")[0];
navigation.onmouseover = function(){headerWrap.style.height = "100px";};
navigation.onmouseout = function(){headerWrap.style.height = "";};
};

Консоль хрома выдает: Uncaught TypeError: Cannot set property 'onmouseover' of undefined

@nubideus · 22.08.2014, 19:49

Denim, не видит navigation(вообще нету в DOM твоего элемента). весь код с html скинь

@Denim · 22.08.2014, 19:54 **[ТС]**

nubideus, спасибо, все работает, опять таки невнимательность моя.

Новые блоги и статьи Все статьи Все блоги /
Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»	SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .
SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .	SDL3 для Web (WebAssembly): Работа со звуком через SDL3_mixer 8Observer8 08.02.2026 Содержание блога Пошагово создадим проект для загрузки звукового файла и воспроизведения звука с помощью библиотеки SDL3_mixer. Звук будет воспроизводиться по клику мышки по холсту на Desktop и по. . .

@Denim 1 / 1 / 1 Регистрация: 05.07.2014 Сообщений: 42
	22.08.2014, 19:19 [ТС]
	quwy, мои знания совсем поверхностные в js. что то не так в условии? 0

quwy Native x86 6857 / 3790 / 1025 Регистрация: 13.02.2013 Сообщений: 11,861
	22.08.2014, 19:36
	Padimanskas, это ж как нужно любить анонимные методы, чтобы использовать их даже там, где сам здравый смысл против. 0

@nubideus 388 / 275 / 76 Регистрация: 19.09.2011 Сообщений: 828
	22.08.2014, 19:42
	quwy, всегда так делаю, систайл. он разве что ; забыл, глаза резануло 0

@nubideus 388 / 275 / 76 Регистрация: 19.09.2011 Сообщений: 828
	22.08.2014, 19:49
	Denim, не видит navigation(вообще нету в DOM твоего элемента). весь код с html скинь 0

@Denim 1 / 1 / 1 Регистрация: 05.07.2014 Сообщений: 42
	22.08.2014, 19:54 [ТС]
	nubideus, спасибо, все работает, опять таки невнимательность моя. 0

Опять таки не понятно почему не работает скрипт

Решение