оценка сайта ,кода

[rootnoob]

Моя первая работа,  изучаю HTML ,CSS месяц ... как улучшить код ? может много классов? должен ли верстальщик (работу делал именно для портфолио ) делать кнопки рабочими и форму? Наверное  не стоит брать заказы , еще поработаю за опыт ..смогу ли я на фрилансе выполнять именно по css и html работу или рынок уже перенасыщен и нужно пробовать с JS что-то? https://jsfiddle.net/rootnoob/fq9tpoy5/ и сам сайт http://business-training.esy.es/

[Mozzie]

Комментарий на глаз:

Визуально мне не нравится оформление формы почему бы не использовать placeholder вместо onfocus, и шапку бы поменьше отступы от лого и саму её меньше. Оживил бы еще кнопки соц сетей :hover к примеру написал и сам сайт не адаптивен. Там где "О бизнес тренинге" как будто картинка съехала вверх и это не красиво смотрится.

[hypnocolor]

Улучшать нужно много чего. Навскидку:

 

• таблицы нужны для табличных данных, вёрстку ими уже давно не делают;

• не задавайте стили с помощью идентификаторов, для этого есть классы (подробнее: http://frontender.info/when-using-ids-can-be-a-pain-in-the-class)

• странные моменты в коде:

<div text-align:left id="header"><img align:"right" src="../head.jpg" class="pictur1">

Очевидно, забыли обернуть стили в style="". Ну и, вероятно, у картинки хотели указать атрибут align, но выглядит это, как css-свойство align, которого не существует. Опять же, стили нужно прописывать в css-файлах за исключением каких-то специфических случаев, когда дико необходимо прописать их в style.

 

Отвечая на вопрос «Должен ли верстальщик делать кнопки рабочими и форму?» — да, должен. Другой вопрос, что под этим имеется ввиду. Если, например, данные формы должны обрабатываться на сервере — это уже задача бэкенда. Хотя умение работать с тем же PHP на базовом уровне для обработки данных, отправки писем и т.д. будет вам только в плюс.

 

JS — нужен в обязательном порядке, просто верстальщик страница сейчас никому не нужен, либо оплачивается очень низко.

 

Вывод: для портфолио это использовать не стоит. По крайней мере, до нормальной доработки.

[Vlad]

<h2><em> - это плохо. <em> нужен для смыслового акцента текста, а не для курсива. Поэтому оставляете один только <h2>, а в стилях ему пишите font-style: italic.

[nomalware]

Да что там про код говорить, смотря на дизайн лендинга, плакать хочется