Просьба прокомментировать

[Den]

Привет. Посмотрите пожалуйста ********

Моё там - это верстка и доработка дизайна, но дизайн меня не интересует. Интересует собственно верстка. Где что не верно, глупо и не правильно, не профессионально, косяки разных браузеров и всё такое. Верстка полностью блочная.

Простите за не отформатированный код, но чистый html где-то уже посеял случайно а движок без шаблонов, там html интегрирован в php и привести код в красивый вид не выходит...

Заранее спасибо

Edited October 29, 2010 by s0rr0w

[Great Rash]

<div class="top"> 
  <div><img src="http://smartdoska.ru/images/news_ico.png" width="16" height="16" alt="" /> 
  <a href="news.html">Новости</a> 
	</div> 
  <div><img src="http://smartdoska.ru/images/img/inf_ico.png" width="16" height="16" alt="" /> 
  <a href="informers.html">Информеры</a></div> 
  <div><img src="http://smartdoska.ru/images/img/advert.gif" width="16" height="16" alt="" /> 
  <a href="contacts.html">Разместить рекламу</a></div> 
  <div class="rb"><img src="http://smartdoska.ru/images/img/contacts_ico.png" width="16" height="16" alt="" /> 
  <a href="contacts.html">Написать письмо</a></div> 

  <div class="city"> <span onclick="window.openCenteredWindow('http://smartdoska.ru/city.html');">Выберите населённый пункт</span> 
	 </div>  
</div>

После такого даже дальше смотреть не стал. Отстой, а не код.

[Den]

Просьба аргументировать свои мнения, ели нет аргументов, не нужно ничего писать.

Спасибо

[Протуберанец]

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

[Den]

Если вы про это:

<span onclick="window.openCenteredWindow('http://****.ru/city.html');">Выберите населённый пункт</span>

То это изначально заложено в движке, в мои задачи не входило копаться в JS. А где стили не вынесены? Я сам не люблю стили в коде.

Edited October 29, 2010 by s0rr0w

[Протуберанец]

width="16" height="16"

[rus]

Простите за не отформатированный код, но чистый html где-то уже посеял случайно а движок без шаблонов, там html интегрирован в php и привести код в красивый вид не выходит...

ну а как вы хотите что б вам отвечали по делу в данном случае?

каков пост - таков ответ.

[Протуберанец]

class="rb" не информативный класс, однако. и чет все дивы да дивы сплошные, гдет слышал что плохой тон, для своих целей испольют свой элементы, а тут все див. "быдлокод" короче как у меня, главное работает

[Den]

Ну что за манера писать и думать, что остальные все поймут...

для своих целей испольют свой элементы

Пример кода покажите, где что, куда?

Если присмотреться, то не сплошные там дивы вовсе, там и других элементов хватает. А дивов много, потому что таблиц нет.

ну а как вы хотите что б вам отвечали по делу в данном случае?

каков пост - таков ответ.

ну там уж не совсем все в кучу и не в одну строку

width="16" height="16"

Лично для меня это очень спорный момент, в том плане, что это нужно было бы вынести в стили.

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

Та я для этого и пишу здесь, чтобы увидеть все свои косяки

Edited October 18, 2010 by Den

[Протуберанец]

Den, сейчас psywalker придет и все тебе расскажет про верстку, а я ламер, мои замечания поверхностны

[Den]

Мне любые замечания важны, так что чем больше критики, тем лучше

[Great Rash]

Просьба аргументировать свои мнения, ели нет аргументов, не нужно ничего писать.

Аргументирую:

Делать меню на дивах - это пипец. Уже по моему все знают, что меню делается списком. Что вам помешало сделать ссылку и с иконки тоже, религия? Может мне по иконкам тыкать нравится. Оформление нужно по возможности отделять от контента, вы же пишете 4 раза подряд width="16" height="16" вместо того чтобы один раз написать в стилях .top img {width: 16px; height: 16px;}. А если вам потом еще пункты в меню понадобится добавить? А если к шаблону придется прикручивать?

Если бы мне как программисту такую работу бы отдал верстальщик, то я бы выяснил где он живет и забил бы ему в голову гвоздь

[rus]

Ну что за манера писать и думать, что остальные все поймут...

Пример кода покажите, где что, куда?

Если присмотреться, то не сплошные там дивы вовсе, там и других элементов хватает. А дивов много, потому что таблиц нет.

ну там уж не совсем все в кучу и не в одну строку

1) манера не лучше чем у вас.

2) пример кода?

вместо:

<div>тут ссылка из меню</div>

написать:

<ul><li>тут ссылка из меню</li></ul>

3) див тоже можно использовать разумно и не нагромождать где только можно.

4) может и не в одну строку, но мы ведь обсуждаем-критикуем не яндекс, или гугл верно?

[Den]

Сначала подумал было отписать свое мнение по этому поводу, но передумал - здесь не спорим а слушаем мнения. Еще есть замечания?

Что вам помешало сделать ссылку и с иконки тоже, религия?

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

PS

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

Будете как все - никто вас и не увидит

Edited October 18, 2010 by Den

[sigma77]

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

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

[Great Rash]

Угу, кагбэ про спрайты слышали?

P.S. Смотрите на монстров (yandex, mail.ru), да хотя б на тот же htmlbook.ru - везде можно тыкнуть как на текст, так и на иконку.

а если заворачивать в одинаковую ссылку отдельно и текст и картинку, то мега-куча лишнего кода

Значит меню манстрячить дивами и прописывать width и height в теле это не мега-куча, а добавить лишний элемент для проставления подчеркивания только тексту - это сразу лишний код?

[Den]

Да, знаю про спрайты

Где я еще налажал?

[Veseloff]

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

[Great Rash]

<form action="http://****.ru/">

Точно валидацию проходит? Вроде как method - обязательный параметр.

Обычно с логотипа стоит ссылка на главную. По крайней мере я так привык.

<div class="divider"></div> 

<div class="main_menu_bg"> 
  <div class="bottom"> 
	<div class="l"></div> 
	<div class="r"></div> 
  </div> 
</div>

Ну это вообще порожняк... я даже не знаю как откомментировать.

<ins class="icon_menu"> 
	  <div> 
		<a href="http://****.ru/"><img src="http://****.ru/images/img/main_img_button.jpg" width="70" height="68" alt="" /></a> 
		<br /> 
		<a href="http://****.ru/">Главная</a> 
	  </div> 
	</ins>

<ins> - inline-элемент, это просто ну ваще... удивительно что оно работает.

<p class="rootcat">Авто. Мото. Транспорт</p>

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

Тег <h1> должен встречаться в коде только раз. Спецификации об этом умалчимают, но любой SEOшник скажет вам об этом (это прям азы SEO).

<div class="divider_col"> </div> 
<div class="col_block"> 
  <ul class="recom"> 
	<li><a href="http://www.****.ru/p12.html">Что такое объявление</a></li> 
	<li><a href="http://www.****.ru/p13.html">Как работать с доской объявлений </a></li> 
	<li><a href="http://www.****.ru/p14.html">Как писать объявление </a></li> 
	<li><a href="http://www.****.ru/p15.html">Тексты объявлений </a></li> 
	<li><a href="http://www.****.ru/p16.html">Доски объявлений рекомендации </a></li> 
  </ul> 
</div>

Чудесно, и этот человек говорит об "мега-куче лишнего кода"!

В общем плохо все.

Edited October 29, 2010 by s0rr0w

[Den]

Ну это вообще порожняк... я даже не знаю как откомментировать

Если вы приведете более простой способ нарисовать закругления углов, я вам спасибо скажу

Обычно с логотипа стоит ссылка на главную. По крайней мере я так привык.

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

По поводу ins

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

По последнему приведенному участку кода я не понял, в чем там проблема.

Edited October 18, 2010 by Den

[Протуберанец]

Если вы приведете более простой способ нарисовать закругления углов, я вам спасибо скажу

http://htmlbook.ru/faq см. раздел "Скругленные уголки".

[Den]

Ни один из вариантов не подходит - опять же лишний код.

Я ж просил не ссылку на faq дать, а привести пример для моего случая

Если еще найдутся конструктивные замечания - буду рад. А пока спасибо, в целом картина ясна.

г-ну. Great Rash, хочу дать совет со стороны. Будьте менее импульсивны и предполагайте хоть иногда, что если так сделано, то может по-другому в этом месте и нельзя было? Старайтесь так же не употреблять в общении грубостей, таких, как пипец, отстой и т.д. Вы все таки модератор - лицо ресурса

Edited October 18, 2010 by Den

[Протуберанец]

Я ж просил не ссылку на faq дать, а привести пример для моего случая

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

странный тип какойт, на подставу похоже ты хто

[Den]

Да нет ничего странного в том, что я использую не 10 строчек кода, а 5, ну что тут странного то? Или странно, что 2 дива завернуты в один родительский, я не пойму?

[Протуберанец]

поведение твое странное и слова и дата реги, все старнное)