Оцените, пожалуйста, верстку- если не затруднит?

Question

[Игорь]

Здравствуйте!
Оцените, пожалуйста, верстку. Буду благодарен, если найдёте время указать ошибки.
Helico_infographic.jpg- файл с макетом в общей папке.

https://github.com/igorde/example.git

Comments

[Сергей delphinpro]

За одни только айдишники - в топку.

[Сергей delphinpro]

Сброс стилей через звездочку - жирный минус. Используйте reset.css / normalize.css
Стили типа "top: -112px;" - снова минус. Это называется "магические числа", коих нужно избегать по максимуму. А при невозможности избежать - обязательно откомментировать, почему здесь такое число. Для позиционирования элементов в современном css есть куча возможностей.
Именование селекторов - тут так, там сяк. Должен быть единый стиль.
class="size_35" - а если нужно будет сделать этот текст размером 36, вы будете писать новый класс size_36 или измените стиль у size_35 (который будет уже не 35)? Нельзя давать такие имена селекторам. Аналогично с классом "white".
Верстка достаточно простая. Уверен, что в более сложной задаче минусов будет намного больше.
Вердикт - учиться, учиться, учиться.

[Игорь]

Спасибо за подробный ответ!

Answer 1

[tema_sun]

Ну, оценивать тут нечего. Методологии вы никакой не придерживаетесь. Зачем-то все сделали на id, почти без классов. Про верхний регистр уже сказали.

Comments

[Игорь]

Спасибо, что ответили! Как считаете: стоит ли мне при текущих навыках в web-студию frontend junior-ом или html-верстальщиком пытаться устроиться? Попытка- не пытка, но совсем глупо не хочется выглядеть.

[tema_sun]

Игорь: современный мир фронтенда это уже давно не только верстка. Как только вы говорите, что хотите занять позицию frontend junior developer от вас сразу будут ждать какое-то знание js и хоть представление о фреймворках. Я не знаю есть ли спрос на верстальщиков сейчас.
Пробовать устроиться конечно нужно. Во-первых, вы узнаете что от вас будут хотеть. Во-вторых, а вдруг возьмут? Где берут джунов обычно есть сеньеры, а где есть сеньеры можно очень быстро и правильно учиться.

[Игорь]

tema_sun: Благодарю!

Answer 2

[lipinart]

Избегайте айдишников, пользуйтесь классами. Изучите принцип БЭМ от Яндекса.
Второе, пользуйтесь сеткой и уделяйте больше внимания типографике, соблюдайте ритм.
Сделайте скрин, наложите на него исходную картинку с 50% прозрачностью и увидите все свои ошибки.

Comments

[Игорь]

Спасибо, что ответили! Как считаете: стоит ли мне при текущих навыках в web-студию frontend junior-ом или html-верстальщиком пытаться устроиться? Попытка- не пытка, но совсем глупо не хочется выглядеть.

[lipinart]

Игорь: Стоит. Лучше попробовать, чем вспоминать, что могло бы быть. К тому же встречаются студии с многолетним опытом делающие куда хуже. Дерзайте, исследуйте, экспериментируйте и все будет. Не все все же стали профи за день

[Игорь]

Понял, благодарю!

[tema_sun]

Я бы тут немного раскрыл мысль - избегайте айдишки для стилизации. Так-то они очень даже нужны.