Jump to content

Оцените верстку III


uteelers5
 Share

Recommended Posts

<input type="text" value="Emailaddress" class="input" />

Замените, пожалуйста value на placeholder. Пусть не кросс браузерно, но лучше пустая строка, чем удалять от туда текст.

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

Edited by alexandr_v-vich
Link to comment
Share on other sites

<input type="text" value="Emailaddress" class="input" />

Замените, пожалуйста value на placeholder. Пусть не кросс браузерно, но лучше пустая строка, чем удалять от туда текст.

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

Честно говоря не могу дать адекватный ответ в сторону табличного меню. Это очень плохо?

Лучше списком или просто ссылки в nav?

Почему меню — таблица? Почему иконки соцсетей 7 картинок, а не одна?

По поводу соц. сетей не понятно, как пользователь попадет в нужную ему соц. сеть если все одной картинкой будет?

Link to comment
Share on other sites

uteelers5,

Честно говоря не могу дать адекватный ответ в сторону табличного меню. Это очень плохо?

Это очень неуместно

Лучше списком или просто ссылки в nav?

Лучше nav. Но он менее кроссбраузерен.

По поводу соц. сетей не понятно, как пользователь попадет в нужную ему соц. сеть если все одной картинкой будет?

Во-первых, 7 больше чем 1 - и это хуже). Во-вторых, создаёте общий блок со ссылками и бэкграундом 1ой картинки. У вас фиксированный дизайн - расположить каждую ссылку под нужную картинку не составит труда :)

Edited by alexandr_v-vich
Link to comment
Share on other sites

uteelers5,

Честно говоря не могу дать адекватный ответ в сторону табличного меню. Это очень плохо?

Это очень неуместно

Лучше списком или просто ссылки в nav?

Лучше nav. Но он менее кроссбраузерен.

По поводу соц. сетей не понятно, как пользователь попадет в нужную ему соц. сеть если все одной картинкой будет?

Во-первых, 7 больше чем 1 - и это хуже). Во-вторых, создаёте общий блок со ссылками и бэкграундом 1ой картинки. У вас фиксированный дизайн - расположить каждую ссылку под нужную картинку не составит труда :)

Спасибо, возьму на вооружение.

Link to comment
Share on other sites

при просмотре на широком экране (даже 1366x768) - фигня

в Google Chrome - полная фигня, не хватает clear:both; после логотипа

http://s017.radikal.ru/i402/1203/b9/a505c6164695.png

текст в меню в макете сероватого цвета, а не абсолютно белый

иконки соц. сетей без сглаживания - видны зубцы

к тому же их(иконки) лучше сделать спрайтом

Edited by NeoXidizer
Link to comment
Share on other sites

к всему сказанному добавлю

1. Safari 5.1.2 - текст в контейнере s-text выходит за нижнюю границу при масштабировании

2. Firefox 11 - изображение baner.png по какой-то причине не загружается и верстка плывет (вид как на изображении из предыдущего сообщения, там тоже это изображение не загрузилось)

3. Нет заголовка h1

Link to comment
Share on other sites

Что за..?

В FF11 многие блоки едут, хотя в других более-менее свежих браузерах (в т.ч. в FF5) такого нет, видимо, это баг новых версий :blink:

P.S. Валидатор ругается на отсутствие alt'ов у изображений и на пустые классы, но это не может сломать вёрстку.

ff11gluk.jpg

Edited by Vin
Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

 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