http://for-test3.narod.ru/

[minelli]

http://for-test3.narod.ru/

Прошу оценить в первую очередь код. Но замечания по общему дизайну так же весьма важны. Ссылки на странице никуда не ведут. Это всего лишь тестовая index.htm. Прошу знающих людей помочь в некоторых вопросах.

[Dimitry Wolotko]

Многие вещи вы делаете на таблице, хотя можно на слоях. Это упрощает редактирование + облегчает код, а следовательно и вес страницы. В частности, говорю про это:

 <table>
 <tr>
 <td>
 <ul class="toolbar">
 <li class="current"><a href="index.php"><b>Главная</b></a></li>
 <li><a href="about.php"><b>О компании</b></a></li>
 <li><a href="news.php"><b>Новости</b></a></li>
 <li><a href="emoloer.php"><b>Работодателям</b></a></li>
 <li><a href="competitor.php"><b>Соискателям</b></a></li>
 <li><a href="contacts.php"><b>Контакты</b></a></li>
 </ul>

 </td>
 </tr>
 </table>

Верстаете на XHTML, а некоторые теги не закрыты. К примеру br.

Вы не умеете работать со стилями, ибо:

 <li><strong>Head-hunting:</strong> прямой поиск кандидатов из числа успешно работающих в настоящее время;</li>

 <li><strong>Approaching:</strong>: выход на конкретных, интересующих Вас лиц;</li>
 <li>Поиск по <strong>Базе Данных</strong></li>
 <li>Поиск по <strong>Интернету:</strong> корпоративный <a href="http://www.wra.com/">веб-сайт</a> и виртуальный HR-офис;</li>

 <li>Привлечение кандидатов через <strong>объявления в СМИ;</strong></li>
 <li><strong>Интервьюирование</strong> кандидатов в офисе "Western Recruitment Agency";</li>
 <li><strong>Тестирование;</strong></li>
 <li><strong>Researching:</strong> подготовка адаптированных резюме с комментариями "Western Recruitment Agency".</li>

 <li><strong>Проверка рекомендаций;</strong></li>

Спокойно исправляет стилями. Не нужно городить такое количество стронгов - это излишне.

Много ляпов, и как мне кажется, из-за них ошибки некоторые и появляются -

- что это такое? Или это -

 Партнеры

- некруто.

Разбирать ваши комментарии в коде я не буду. Можете на каждый интересующий вопрос создать топик в соответствующих разделах.

[minelli]

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

[Dimitry Wolotko]

На вкус и цвет.

[minelli]

по XHTML и h3 абсолютно верное замечание. правится легко, исправлю. а вот по стронгам...

<li><strong>Approaching:</strong>: текст без стиля

менять на на нечто...

<li><тег class="такой то">Approaching:</тег>: текст без стиля

а смысл? в странице идет чистый html, тег стронг задан глобальным стилем.

или я неправильно понял замечание?

[homm]

а смысл? в странице идет чистый html, тег стронг задан глобальным стилем.

или я неправильно понял замечание?

Правильно. Я с тобой тоже согласен. И Владимир Токмаков с нами согласен.

[minelli]

2 Dimitry за

<DIV class"boxed"> - что это такое?

отдельное спасибо. я долго искал эту блоху ночью.

[homm]

В фф шапка более мнее нормально (если крестики посреди шапок можно считать нормальным), но в футере появилось?

[homm]

Разрывы от того, что для таблицы нужно указывать cellpadding cellspacing

Флоат элементы не расширяют высоту родителей. Ставте в конце
или аналог.

Почему бы клас ?boxed? не приписать к адресу? Зачем там див?

[homm]

[*]Главная |

[*]О компании |

[*]Новости |

[*]Работодателям |

[*]Соискателям |

[*]Контакты

Наверное так?

<a href="index.php">Главная</a> | <a href="about.php">О компании</a> | <a href="news.php">Новости</a> | <a href="emoloer.php">Работодателям</a> | <a href="competitor.php">Соискателям</a> | <a href="contacts.php">Контакты</a>

А если кто скажет, что не по-пацански, бей с левой.

тока убери

[minelli]

так и было. захотел списком, чтобы если вдруг у меня получится, то мог быстро менять 2 списка с меню (первый в тулбаре). они ведь идентичные. только стили разные. погоня за расширяемостью. типа, поменял графику, меню, расположение блоков и вот новый шаблон! может и лишнее.

[minelli]

у меня задают стиль картинки на :hover в тулбаре. без них корректно не отображаеся.

menu. css

.toolbar li {float:left;}
.toolbar li a {display:block; float:left; height:35px; line-height:35px;
 color:#AAA; text-decoration:none; font-size:11px; font-family:arial, verdana, sans-serif; font-weight:bold;
 text-align:center; padding:0 0 0 8px; cursor:pointer;white-space: nowrap}
.toolbar li a b {float:left; display:block; padding:0 16px 0 8px;white-space: nowrap}
.toolbar li.current a {color:#EE5F00; background:url(../img/button3.gif);}
.toolbar li.current a b {background:url(../img/button3.gif) no-repeat right top;}
.toolbar li a:hover {color:#FFF; background:#000 url(../img/button2.gif);}
.toolbar li a:hover b {background:url(../img/button2.gif) no-repeat right top;}
.toolbar li.current a:hover {color:#EE5F00; background:#000 url(../img/button3.gif); cursor:default;}
.toolbar li.current a:hover b {background:url(../img/button3.gif) no-repeat right top;}
table {width: 860px; height:35px; padding:0; margin:0; border:0;}

[homm]

Почему, кстати в шапке див а не img?

[minelli]

Почему, кстати в шапке див а не img?

попробовал. вернулся к исходному. блоки прописаны в blocks.css и ИМХО, так я имею больше гибкости. то есть могу передвигать дивы как слои в фотошопе.

blocks.css

#container {width:860px; background-color:#FDFDFD; margin-left:auto; margin-right:auto;}

 #header {width:860px; height:290px; clear:both;}
 .logo {float:left; width:505px; height:290px; background: url(../img/logo.jpg) no-repeat;}
 .picture {float:right; width:355px; height:290px; background: url(../img/picture.jpg) no-repeat;}
 .toolbar {padding:0 0 0 32px; margin:0; height:35px; position:relative; background:#FFF url(../img/button1.gif);
 border:1px solid #000; border-width:0 1px; border-bottom:1px solid #444;}

 #content {width:860px; padding-top:15px;}

 #main {float:left; width:600px;}
 #sidebar {float:right; width:220px; padding-top:10px;}

 #footer {font-size:11px; text-align:center; width:860px; height:35px; position:relative; list-style:none; margin:0;}

[minelli]

Флоат элементы не расширяют высоту родителей.

добавил класс boxed к тегу address и IE 6 заполнил фоновым цветом все. без
а в опере действительно высота не расширилась.

[minelli]

Не нужно городить такое количество стронгов - это излишне.

текст на странице - целиком и полностью предоставлен заказчиком. исправил только некоторую пунктуацию, типа пробелов.

[minelli]

Незачем через слово указывать людям что делать и считать что у них к вам особое отношение, что вас на смех хотят поднять. Слишком мелкая сошка.

недоразумения выяснены. я почистил топик от ненужных символов. предлагаю сделать тоже самое :-)

[minelli]

Разрывы от того, что для таблицы нужно указывать cellpadding cellspacing

да, действительно, так и есть. а разве

table {padding:0; margin:0;}

не равняется

<table cellpadding="0" cellspacing="0">?

[Sotnikov]

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

уже вижу, что в классе .toolbar { нужно поставить list-style: none;

а

[minelli]

.toolbar {list-style: none;

в принципе список прячется за бэкграундом таблицы и маркеров не видно. но проставил. спасибо.

теперь еще раз объяснюсь, почему залез в этот раздел. я нуб в дизайне и не скрываю этого. задавал вопросы в разделе для начинающих. скажу, что спецы не слишком снисходят до таких, как я. в итоге ZoNt пишет - "... чувак, залей на бесплатный, глянем, поправим." согласен, с народом лоханулся. гуглю постоянно :-) очень многие параметры нахожу методом тыка. нужное запоминаю, а вот что касается общего принципа... для этого и повесил сюда. лучше попасть на тяжеловесов, конкретно выгребсти, но получить урок и качать мышцу. как в жизни. разве не так?

[minelli]

Логотип кстати я где-то в видел в шаблонах...

вот он

с логотипом целая эпопея. но если заказчику понравилось нечто, напоминающее лодку на парусах, то пусть так и будет. можно, пользуясь уроками фотошоп нарисовать эти самые паруса. но можно и просто взять их из шаблона.

[minelli]

а девочка в шапке взята из клипарта 5-7 летней давности. у нее мониторчик правда другой был :-)

[Vlad]

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

Да..., и это..., картинку бы надо получше качеством поставить или вообще заменить.

[minelli]

спасибо. графика пока вообще не при делах. основной упор был на отладку кода, чтобы можно было быстро переделывать шаблоны. менять местами блоки, графику, меню.

[minelli]

пока можно прихлопнуть этот топик.