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

[Zverushka]

vtest.uphero.com/wp/

psd шаблон http://vtest.uphero....psdtemplate.psd

Его скриншот http://vtest.uphero....psdtemplate.jpg

[saiidr]

Вроде вёрстка хорошая =) У меня вроде багов не было...

Заметил только,что на кнопке (Submit) в низу при наведении,предоставляет вставить курсор. Можно было бы сделать cursor: pointer;,что было понятно,что ты нажимаешь на данную кнопку =)

Edited July 24, 2013 by saiidr

[T001]

vtest.uphero.com/wp/

psd шаблон http://vtest.uphero....psdtemplate.psd

Его скриншот http://vtest.uphero....psdtemplate.jpg

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

[Zverushka]

У меня ушло полтора полных дня (день 8+ часов), особенно много времени потеряла на поиске слайдера и подделки под него.

[SelenIT]

Можно было бы сделать cursor: pointer;,что было понятно,что ты нажимаешь на данную кнопку =)

Лучше всё-таки реализовать для нее полноценный :hover. Показывать кликабельность одной сменой курсора... ну как-то несолидно, имхо. Всё-таки изначально и по стандарту pointer для ссылок и только для них. Да и юзабилити-эксперты призывают не полагаться на курсор мыши как «индикатор кликабельности»:

Users should not be required to use the mouse pointer to discover clickable areas of images.

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

There is a visible change when the mouse points at something clickable (excluding cursor changes).

Присутствует видимое изменение, когда мышь указывает на что-то кликабельное (помимо изменения курсора).

[Vulner]

Думаю можно было бы объединить

<link href="css/reset.css" rel="stylesheet">
<link href="css/style.css" rel="stylesheet">

в один файл. Одним запросом меньше

Создавать для тени отдельные элементы

<div class="footer-shadow-top"> </div>
<div class="footer-shadow-bottom"> </div>

не есть хорошо. Лучше их сделать через footer:after, footer:before. Кстати сам футер неплохо было бы прижать к низу

И по поводу кнопки DOWNLOAD NOW. Наверное лучше чтобы это была не картинка. А блочок с заливкой и текстом.

[Zverushka]

@Vulner,

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

А по поводу бефо и афте в футере. Дело в том, что бэкграунд по задумке белый и при вставке теней через псевдо-элементы полупрозрачные тени оказались бы на белом. Или пришлось бы внутрь футера вставлять еще один див с белым фоном...

[SelenIT]

и при вставке теней через псевдо-элементы полупрозрачные тени оказались бы на белом

Но ведь можно задать для них и background-color, и background-image?

[Zverushka]

ну, background color должен быть прозрачным, а под прозрачным окажется белый цвет футера, а background-image и есть сама тень.

[SelenIT]

background color должен быть прозрачным

Кому должен? Кто запретит ему слиться со светло-серым основным фоном, например? Эх, жалко, что :after от .wrap уже занят, на мой взгляд, эта тень в него напрашивается...

[Zverushka]

На случай, если захочется другой background у страницы сделать, чтобы не пришлось править еще другие элементы, чтобы сайт был "масштабируемым" изначально.

[SelenIT]

В принципе, логично. Тем более аргумент за то, чтобы эта тень не была привязана к белому футеру. Может, всё-таки ее на .wrap:after повесить, а боковые тенюшки к .top-у прицепить, с которым они и визуально связаны?

[Vulner]

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

В том виде, в каком оно все сейчас, так оно и выходит. Но исходя из того,

чтобы сайт был "масштабируемым" изначально.

то нужно его прижать. Ведь если убрать контент, то где окажется футер? Верно, не в низу страницы, где ему и место, а где-то по серединке.