Верстка в первый раз. Ну как?

Question

[colorkid]

Хочу сменить профиль с менеджера проектов на front-end разработчика.
Сейчас изучаю верстку через htmlacademy и htmlbook.
Вот на работе появилась задача сделать сайт по недвижки, решил сверстать сам. colorkid.ru.swtest.ru (colorkid.ru)
Прошу оценить как для первого раза? Допустил ли я какие нибудь фатальные ошибки?
Может рекомендацию дадите?

Answer 1

[Сергей]

1. <meta="utf-8"> - не правильный тег. Правильно: <meta charset="utf-8">
2. Далее неплохо было бы указать <meta name="description" content=""> Так как он учитывается поисковиком.
3. Ссылкам и изображениям необходимо прописывать title и alt. Почитайте про их употребление.
4. Старайтесь избегать смешивания чисто английских названий классов и русских транскрипций навроде <ul class="categoriimenu">. Стремитесь к единообразию структуры.
5. Используйте семантические теги html5 - header, footer, section, nav и так далее.
6. В целом для первого раза - неплохо. Удачи!

Comments

[colorkid]

спасибо

Answer 2

[dm]

1. Семантика на нуле. Header, footer, nav и пр. отсутствуют
2. Названия классов есть плохие (title, name, ten и пр.) – пространство имен и все такое. Ten – по названию нифига непонятно к чему это может относиться.
3. Пункты меню в html написаны капслоком, а если передумает владелец сайта, нужно будет менять текст. Есть же text-transform: uppercase
4. В css если значение 0, то пиксели можно не писать.
5. Валидатор - validator.w3.org/check?uri=http%3A%2F%2Fcolorkid.r...
6. padding-top:5px; padding-bottom:5px; Такое можно писать короче - padding: 5px 0;

Comments

[colorkid]

Спасибо за комментарии

Answer 3

[Евгений Бухарев]

1. meta="utf-8"
2. Есть вложения блочных элементов в строчные (a>div)
3. Агенство -> Агентство : )
Ну а в целом для первого раза довольно не плохо

Comments

[colorkid]

благодарю за отзыв.

[Евгений Петров]

evgenybuckharev В HTML5 в ссылках может находиться любой контент. Блочный, строчный, любой. А вот от содержания ссылки уже зависит, в каком контексте может появиться она. Называется такой тип контекста transparent,
RTFM

Answer 4

[Антон Мудренок]

вот хочу я переслать своей девушке цену за квадрат и че мне делать, руками набирать?)
еще и подчеркнули текст как ссылку, за что так издеваться...

Comments

[Сергей]

Ну это можно отнести к недостатку дизайн-макета, а не верстки :)

[Антон Мудренок]

Hando: да ладно? то что текст нельзя скопировать бока макета? :D

[Сергей]

Антон Мудренок: я про подчеркнутый текст, что он тпо ссылкой не является. То что текст нельзя скопировать - согласен, косяк)

[Антон Мудренок]

Hando: да тут таких косяков... если бы человек не сказал, что первый раз, можно было бы палец отрезать)

[colorkid]

всем спасибо за мнения и отзывы

Answer 5

[Андрей Федоров]

1. Статические сайты - это неплохо. Но и не хорошо. Нужно изучать php и сразу делать cms. Чтобы клиент вас не дергал потом по каждому чиху.
2. Отсутствие адаптивности. Для первого раза норм, но сделали и забудьте. Сайт нужно продумывать заранее так, чтобы с адаптивностью не было никаких проблем. Иначе потом, когда он обрастет фишками и плюшкам, вас попросят внедрить адаптивность и вам придется ПЕРЕДЕЛЫВАТЬ АБСОЛЮТНО ВСЕ!!! Это сильнейший удар по мотивации.

Comments

[colorkid]

благодарю

[dpismenny]

Статические сайты плохо? Ну это ж очень от команды зависит. Есть команды у которых свой фреймворк, и им нужна только верстка. Даже не gulp, bower, sass, etc. А просто html/css/js. Даже JS многие команды после окончания верстки потом переписывают. Так что это очень хорошо для фрилансера-дженералиста или если в команде нужен такой специалист :)

[Андрей Федоров]

Читаем внимательно, да? )