Jump to content

Оценить менюшку.


boom.olezka
 Share

Recommended Posts

ну начнем с того, что такие вещи давно и довольно успешно делаются с помощью одного только CSS :)

постараюсь немножко покритиковать :)

1. не надо везде прописывать onmouseover="showmenu(3); и т.п. Все это можно сделать с помощью банального цикла

2. както здесь плохо названа переменная ибо метод getElementById возвращает всего одну ссылку на объект, а не несколько

var allPageTags=document.getElementById('menu_id_' + id);

3. не нравится такая конструкция onmouseover="dont_hide_menu(5);" onmouseout="hide_menu(5);". Получается, что менюшка сначала прячется, а потом сразу же появляется. надо сделать по другому... как подумай сам

4.

лучше не использовать. можно просто сделать ссылки блочными. будет намного лучше

5. ну и т.д. в таком же духе :)

Link to comment
Share on other sites

Guest
This topic is now closed to further replies.
 Share

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. See more about our Guidelines and Privacy Policy