Skip to content

Conversation

@legitttt1
Copy link

Простите меня пожалуйста! Я закинул лабу в срок, коммит от 20 ноября, просто не создал новую ветку и пулл реквест!

Copy link

@mxwrlld mxwrlld left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В целом, хорошо!

  • Добавьте адаптивности, например, с помощью медиа - запросов: пусть при просмотре с узких экранов мобильных устройств блок с фотографией и персональной информацией располагается вверху страницы, а блок main-content под ним
  • Согласуйте имена классов с БЭМ. Напоминаю, соглсно БЭМ именуем классы по шаблону имя-блока__имя-элемента_имя-модификатора. Подробнее тут и тут. Сейчас нет ни одного элемента, сплошные блоки. Например, логично сделать навыки элементами блока skills, т.е. skills__item вместо skill-tag. soft становится модификатором элемента навыка, т.е. skills__item_soft и т.д.

Copy link

@mxwrlld mxwrlld left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Кажется, это всё ещё не то что нужно

image image

@mxwrlld
Copy link

mxwrlld commented Dec 18, 2025

Зачтено!

@mxwrlld mxwrlld closed this Dec 18, 2025
@github-actions github-actions bot added Done and removed In progress labels Dec 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants