Skip to content

Conversation

@YaroslavBonzo
Copy link

No description provided.

@YaroslavBonzo YaroslavBonzo changed the title 6212 ЩукинЯВ Лаб.4 6212 ЩукинЯВ Лаб.3 Nov 17, 2025
@AvtoBBus AvtoBBus requested a review from mxwrlld November 17, 2025 09:55
@AvtoBBus AvtoBBus added Lab 3 and removed Lab 4 labels Nov 24, 2025
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

index.html Outdated
<meta charset="UTF-8">
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<title>Резюме</title>
<link rel="stylesheet" href="style/style.css">
Copy link

Choose a reason for hiding this comment

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

Ссылка некорректная

index.html Outdated
<header class="resume__header">
<div class="resume__photo-section">
<img
src="style/images.jpg"
Copy link

Choose a reason for hiding this comment

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

Ссылка некорректная + изображения лучше размещать в папках с изображениями, вместо style

index.html Outdated
Comment on lines 53 to 58
<div class="skills">
<span class="skill">C++</span>
<span class="skill">HTML</span>
<span class="skill">CSS</span>
<span class="skill">Python</span>
</div>
Copy link

Choose a reason for hiding this comment

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

По БЭМ тут более подходящим будет вариант skills__item, как и в блоке хобби

@YaroslavBonzo
Copy link
Author

исправил

@mxwrlld
Copy link

mxwrlld commented Dec 1, 2025

Исправьте ссылки на CSS - файл и аватарку. Все ещё некорректны. В репозитории нет папки styles на которую ссылается HTML
image

В остальном все хорошо, остальные замечания учтены

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.

Напоминаю о необходимости внесения изменений!

@YaroslavBonzo
Copy link
Author

Сделал

@mxwrlld
Copy link

mxwrlld commented Dec 19, 2025

Ура! Зачтено!

@mxwrlld mxwrlld closed this Dec 19, 2025
@github-actions github-actions bot added Done and removed In progress labels Dec 19, 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.

3 participants