оценить верстку

[pupil]

прошу оценить верстку(кликабельно)

[121990]

Быстра глянул. Слайдер не адаптивный. Стрелки танцуют до размера 1280x800 и то перекрываются слайдером. От 1024x768 - так вообще пропадают. Плюс в инпутах не выравнен текст по высоте. Style.css не читабельная абра-кадабра.

Edited August 6, 2013 by 121990

[Struggle]

Ужас, переделывайте, не читабельно буквально всё, франкенштейн какой-то

Edited August 6, 2013 by Struggle

[pupil]

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

стрелки "достал", но с их плясками не разобрался, посмотрю утром, как и инпуты.

style не понравился в том плане, что описание селекторов в одну строку?

[Struggle]

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

стрелки "достал", но с их плясками не разобрался, посмотрю утром, как и инпуты.

style не понравился в том плане, что описание селекторов в одну строку?

В одну строку хорошо, только делать это нужно когда сдаешь проект, а тут мы должны портить себе глаза искать что либо, так что приведи в порядок код, да и вообще не хорошо делать в 1 строку, т.к если твой проект будут еще пилить, верстальщик прикурит

[121990]

что описание селекторов в одну строку?

Лучше всегда в столбик оформлять стили. Я по-разному делал, но этот вариант лучший. Почему? думаю комментировать не стоит.

Edited August 6, 2013 by 121990

[pupil]

Ужас, переделывайте, не читабельно буквально всё, франкенштейн какой-то

вас так смутила разметка или css?