• Оцените верстку начинающего верстальщика?

    Qairat
    @Qairat
    frontend developer, angular 2+
    Сразу начинай изучать scss, забей на css. Вот маленький пример с твоего кода, как можно написать на scss.
    $green: #1abc9c; // можешь потом везде его использовать, и в других файлах scsc.
    $green2: #18a68a;
    $blue: #3498db;
    $blue2: #2e85bf;
    .button { // дай общий класс для двух div-ок
    	padding: 15px 0px;
    	border-radius: 4px;
    	border: none;
    	color:#fff;
    	cursor: pointer;
    	font: 14pt SlimSansSerif;
    	width: 166px;
    }
    .btn1{
    	background-color: &green;
            &:hover{
    	      background-color: &green2;
            }
    }
    .btn2{
    	background-color: $blue;
            &:hover{
              	background-color: &blue2;
            }
    }


    И еще, пиши в отдельном файле все media стили.
    Ответ написан
    2 комментария
  • Оцените верстку начинающего верстальщика?

    sfi0zy
    @sfi0zy Куратор тега CSS
    Creative frontend developer
    По внешнему виду:
    Адаптивность есть, это хорошо. Есть мелкие проблемы, которые портят впечатление (где-то что-то обломалось, где-то что-то не влезло):
    240b345906f84e7ca5af8252d9b00964.png51d00024eb5c4c8896e8e888ce8ea000.png821cc5a47651408c8728229982fe1521.png
    и.т.д. Также стрелка снизу кажется не по центру, отступы какие-то странные местами (макет не смотрел, это чисто личное впечатление). Фокус на элементах практически не заметен. Стоит поработать над этим. Обводка синим - бяка.

    По коду:
    - Отсутствует система в именовании классов. Почитайне про БЭМ или RSCSS.
    - Использовать #id в CSS не стоит. Это дурной тон и источник проблем.
    - Из-за вышеупомянутого отсутствия системы классов CSS читать невозможно + потенциальные проблемы с перебиванием стилей в неожиданных местах.
    - Много магических чисел в CSS (321px, 44.148936%, 381px...). Практически всегда можно избежать этого.
    - Нет ресета/нормалайза. Тоже потенциальные проблемы.

    Поправьте вышеупомянутые проблемы. После этого приходите к использованию препроцессоров (можно начать с LESS - он самый простой, но в 99% случаев его возможностей достаточно). Ну и JS изучайте, без него верстальщику не выжить.
    Ответ написан
    1 комментарий