Оцените верстку пожалуйста.

brevis

Well-Known Member
Регистрация
10.08.2010
Сообщения
452
Годная
Очень
Верстка
Нечего
Обо**ать

это просто что бы было смешно
а так наверное хорошая верстка (я в этом ничего не понимаю), молодец
 

Volly

Well-Known Member
Регистрация
21.03.2012
Сообщения
831
Замечания на прилагаемой картинке
 

Вложения

stan077

Well-Known Member
Регистрация
01.10.2012
Сообщения
108
[member=Volly],
Cпасибо, учту.
 

Maxchen

Well-Known Member
Регистрация
03.03.2011
Сообщения
64
Вёрстка очень даже ничего! Не знаю, что добавить по вопросу, но хочется отметить, что блок слева от логотипа какбы выглядит лишним.
На мой взгляд можно было бы всё это вверху сжать и разместить сбоку в панельке. Но это моё мнение, автор решает сам ;)
 

stan077

Well-Known Member
Регистрация
01.10.2012
Сообщения
108
[member=Maxchen],
Cпасибо, за комментарий
 

BaNru

Пацифизжу
Команда форума
Регистрация
13.11.2010
Сообщения
4 137
Не смотрел особо, ибо лень, но сразу бросилось в глаза
- Logo под картинкой, вместо названия
<span>Logo</span>- Фон картинкой, а не стилями
Код:
<img class="head_img" src="images/bg.jpg" alt="картинка">
 

stan077

Well-Known Member
Регистрация
01.10.2012
Сообщения
108
BaNru сказал(а):
- Logo под картинкой, вместо названия
<span>Logo</span>
[color=rgb(40,40,40);font-family:helvetica, arial, sans-serif;]Извините под какой картинкой?? У меня лого бекграундом прописано , а <span>Logo</span> прописаночтобы валидатор на пустой h1 не ругался.[/color]


- Фон картинкой, а не стилями
[color=rgb(0,0,136);]<img[/color] [color=rgb(102,0,102);]class[/color][color=rgb(102,102,0);]=[/color][color=rgb(0,136,0);]"head_img"[/color] [color=rgb(102,0,102);]src[/color][color=rgb(102,102,0);]=[/color][color=rgb(0,136,0);]"images/bg.jpg"[/color] [color=rgb(102,0,102);]alt[/color][color=rgb(102,102,0);]=[/color][color=rgb(0,136,0);]"картинка"[/color][color=rgb(0,0,136);]>[/color]
Это резиновая верстка и картинку проще растянуть чем бекграунд ;)
 

BaNru

Пацифизжу
Команда форума
Регистрация
13.11.2010
Сообщения
4 137

lekzd

parse error: parse error, unexpected T_STRING...
Регистрация
17.02.2011
Сообщения
1 125
http://screenshot.su/show.php?img=cf56cd77bc53ae5ee76d854d244ec75d.jpg
надо улучшать интерфейсы грамотно
1, 2, 3: нету отсутпов справа-слева
4: не нужно делать фон у селекта, если не заменять его полностью, кроссбраузерно не будет никогда
5: надпись "найти" на кнопке как-то криво расположена
http://screenshot.su/show.php?img=c30e51c7166b8a9897a641b7697120cc.jpg
хотелось бы чтобы это был сабмит, как и везде, люди привыкают к хорошему, не надо придумывать что-то новое
http://screenshot.su/show.php?img=e440167c5226557678c8efd98a997767.jpg
лучше не надо использовать картинки как элементы оформления, это плохо для SEO
раз уж начали использовать HTML5, то оборачивайте блоки ссылок в <nav>

http://screenshot.su/show.php?img=0d427abea349129f460a6fc079512260.jpg
определитесь с величинами, при резиновой верстке проценты + пиксели дают не всегда ожидаемые результаты, либо используйте box-sizing: border-box чтобы padding не расширял блок


http://screenshot.su/show.php?img=88a8193d7f8e93096478c06a34826054.jpg
выучите что такое и зачем нужен clearfix чтобы не пришлось любыми способами отрывать блоки друг от друга
http://screenshot.su/show.php?img=4799a1109da4332d75adbad715035cf4.jpg
зачем писать капсом, если на это есть CSS http://htmlbook.ru/css/text-transform
А лого сайта лучше картинкой, нежели элементом с фоном
 

stan077

Well-Known Member
Регистрация
01.10.2012
Сообщения
108
[member=BaNru],
[color=rgb(40,40,40);font-family:helvetica, arial, sans-serif;]А попробовать написать название фирмы не айс? не айс[/color]
[color=rgb(40,40,40);font-family:helvetica, arial, sans-serif;]на счет [/color]
background-size можно конечно, но его ие8 и 9 не понимают
 

lekzd

parse error: parse error, unexpected T_STRING...
Регистрация
17.02.2011
Сообщения
1 125
stan077 сказал(а):
но его ие8 и 9 не понимают
чего? http://screenshot.su/show.php?img=0566ffed28e9fa5c0260663b972b32f0.jpg


stan077 сказал(а):
А попробовать написать название фирмы не айс? не айс
а зачем тогда H1 ? это же главный заголовок страницы, самое то использовать его для заголовка раздела а не для логотипа
 

stan077

Well-Known Member
Регистрация
01.10.2012
Сообщения
108
lekzd[color=rgb(40,40,40);font-family:helvetica, arial, sans-serif;]раз уж начали использовать HTML5, то оборачивайте блоки ссылок в <nav>[/color]
[color=rgb(40,40,40);font-family:helvetica, arial, sans-serif;]я читал что тег [/color][color=rgb(40,40,40);font-family:helvetica, arial, sans-serif;]<nav> должен быть только один но странице [/color]
 

stan077

Well-Known Member
Регистрация
01.10.2012
Сообщения
108
[member=lekzd],
Спасибо,буду знать
 

stan077

Well-Known Member
Регистрация
01.10.2012
Сообщения
108
[member=BaNru],
Можно, но ето все-равно придется вырезать фон подгонять текст (а еще если он костомный искать в инете...) вобшем лишний гемор :blink:
 

lekzd

parse error: parse error, unexpected T_STRING...
Регистрация
17.02.2011
Сообщения
1 125
stan077 сказал(а):
вобшем лишний гемор
зачем вам вообще учиться верстать? работать поваром куда проще:
с утра порезал заготовки, разобрал заморозку
целый день сидишь открываешь заготовки и готовишь по готовым схемам, вечером всегда в одно время уходишь домой, можно брать ипотеки, кредиты и содержать семью
 

BaNru

Пацифизжу
Команда форума
Регистрация
13.11.2010
Сообщения
4 137
stan077 сказал(а):
Можно, но ето все-равно придется вырезать фон подгонять текст (а еще если он костомный искать в инете...) вобшем лишний гемор :blink:
Какой фон, какой текст? Т.е. для слова "LOGO" в теге H1 мы этого (выше описанного) не делали, а вот чтобы написать семантичносеошно название сайта/фирмы, у нас появляется геморрой?!

Объясню прямым текстом
Открываем исходник верстки и находим там
Код:
<h1 class="logo"><a href="index.html" class="logo_link" title="На главную"><span>Logo</span></a></h1>
Находим слово "Logo" и пишем вместо него название сайта
Код:
<h1 class="logo"><a href="index.html" class="logo_link" title="На главную"><span>Baha Turaev's Design</span></a></h1>
Profit
 
Верх Низ