Привет! Оцените пожалуйста верстку, что стоит изменить или добавить?
Результат veles: https://greedmagic.github.io/veles/build/
Если не секрет, то сколько занимаетесь фронт-эндом и проходили ли какие либо курсы?
Проходил начальный курс по верстке и js в конце 18 года, закрепить знания на практике не удалось тк основная работа отнимала все силы. Начал практиковаться в начале декабря 19 года.
Любая оценка субъективна, поэтому не обязательно все комментарии будут справедливыми.
Смотрела очень быстро.
Что понравилось:
- Внешне выглядит довольно чисто. Но лучше, конечно, взглянуть на jpg чтобы сказать точно.
- Сделаны взаимодействия. Причем, скорее всего, многое придумывали сами.
- Есть адаптивность, при ресайзе не возникает прокрутки.
- Понятное именование классов (почти везде).
Что можно улучшить:
- Посмотреть как выглядит страница при загрузке. Прыгает высота витрины - не очень красиво. И вообще все слайдеры прыгают при загрузке. Посмотрите как грузятся известные сайты со слайдерами. На яндексе, например, ничего не прыгает.
- Посмотреть блоки на переполнение. Например, если добавить текста в маленькие блоки новостей в правой колонке, то уезжает текст в большом блоке. У первой большой новости не очень понятный ховер (возможно, проблема макета).
- При ресайзе пропадает подвал. На планшете нет кнопки закрытия меню. Возможно, проблема в макете.
- Опечатка в классе random-article__butoon - button
- По дефолту у кнопки type=“submit”, если она ничего не отправляет, то лучше прописать type=“button” (на интенсивах обычно просят в этом случае указать тип явно)
- Если с иконок не планируется ссылка на страницу, то наверное они - кнопки. Здесь исправлюсь. Скорее всего, вы правы и в случае отсутствия js можно будет организовать открытие ссылки. Т.ч. надежнее - ссылки.
- В ie11 совсем сломалось. Если ломается, то пусть ломается без вреда для контента, чтобы его можно было прочитать. Сейчас нельзя.
- mm-spn–light mm-spn–navbar mm-spn–main mm-spn - ась? тут в вас временно сломался студент академии. Вы вполне квалифицированы, чтобы сверстать без доп. библиотек (ну разве что слайдеры понадобятся). Внешне в макете нет сложностей.
- main-menu__link is-current - а как же БЭМ?
- внизу странный js с //
- в app.js лишний код https://greedmagic.github.io/veles/build/js/app.js
- Хорошо бы делать второй файл стилей, который не минифицирован, чтобы удобно было его смотреть.
- Мне кажется, кнопки можно не ограничивать вот так max-width: 170px; - возникнет ситуация когда это может помешать.
- Наверное, состояние наведения (и вообще кликабельность) можно делать для всей новости - выводить оверлей над фото, показывать иконки и подсвечивать заголовок. Не вполне очевидно, что при наведении на фото есть доп. опции.
В целом впечатление хорошее, положительное. Можно идти на продвинутый курс по верстке, если есть такие планы. Ну и совершенствовать js, выходить на более серьезный уровень.
1 лайк