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

stan077

Well-Known Member
Регистрация
01.10.2012
Сообщения
108
[color=rgb(40,40,40);font-family:helvetica, arial, sans-serif;]Добрый день.[/color]
[color=rgb(40,40,40);font-family:helvetica, arial, sans-serif;]Оцените пожалуйста верстку, укажите на ошибки кода.[/color]
Для просмотра ссылки необходимо: Войти или Регистрация
 

brevis

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

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

Volly

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

Вложения

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 020
Не смотрел особо, ибо лень, но сразу бросилось в глаза
- 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 020
stan077 сказал(а):
а Logo прописаночтобы валидатор на пустой h1 не ругался.
А попробовать написать название фирмы не айс?

stan077 сказал(а):
Это резиновая верстка и картинку проще растянуть чем бекграунд ;)
Для просмотра ссылки необходимо: Войти или Регистрация
 

lekzd

parse error: parse error, unexpected T_STRING...
Регистрация
17.02.2011
Сообщения
1 125
Для просмотра ссылки необходимо: Войти или Регистрация
надо улучшать интерфейсы грамотно
1, 2, 3: нету отсутпов справа-слева
4: не нужно делать фон у селекта, если не заменять его полностью, кроссбраузерно не будет никогда
5: надпись "найти" на кнопке как-то криво расположена
Для просмотра ссылки необходимо: Войти или Регистрация
хотелось бы чтобы это был сабмит, как и везде, люди привыкают к хорошему, не надо придумывать что-то новое
Для просмотра ссылки необходимо: Войти или Регистрация
лучше не надо использовать картинки как элементы оформления, это плохо для SEO
раз уж начали использовать HTML5, то оборачивайте блоки ссылок в <nav>

Для просмотра ссылки необходимо: Войти или Регистрация
определитесь с величинами, при резиновой верстке проценты + пиксели дают не всегда ожидаемые результаты, либо используйте box-sizing: border-box чтобы padding не расширял блок


Для просмотра ссылки необходимо: Войти или Регистрация
выучите что такое и зачем нужен clearfix чтобы не пришлось любыми способами отрывать блоки друг от друга
Для просмотра ссылки необходимо: Войти или Регистрация
зачем писать капсом, если на это есть CSS Для просмотра ссылки необходимо: Войти или Регистрация
А лого сайта лучше картинкой, нежели элементом с фоном
 

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 не понимают
чего? Для просмотра ссылки необходимо: Войти или Регистрация


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

stan077

Well-Known Member
Регистрация
01.10.2012
Сообщения
108
Для просмотра ссылки необходимо: Войти или Регистрация[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]
 

lekzd

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

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 020
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
 
Верх Низ