Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add task solution #4697

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

Teslaanazuma
Copy link

Сброс полей браузера по умолчанию.
используйте шрифт Roboto
Используйте семантические теги: , , , , и


  • Заголовок должен растягиваться на всю ширину страницы
    Высота должна быть задана для навигационных ссылок (а не для шапки), берите ее из дизайна.
    Содержимое заголовка должно быть вертикально центрировано.
    Логотип также должен быть ссылкой с изображением внутри (из src/images). Но он не должен быть частью .
    ❗️ Синяя ссылка со строкой ниже должна содержать класс в дополнение к любым другим классам, которые вы добавляете.is-active
    ❗️ Добавьте атрибут к 4-й ссылке для тестирования (data-qa="hover"Laptops & computers)
    Цвет ссылки должен быть изменен на :hover
    Используйте метод и расположите его относительно ссылки с классом::afteris-active
    Не используйте свойство flex для отступов. Он не поддерживается в тестах, используйте вместо этого.gapmargin
    Не должно быть полей перед первым и после последнего элемента списка
    Не стоит просто копировать все стили из Figma. Подумайте, какие из них актуальны. Неравномерные размеры (например, ) определенно бесполезны.line-height: 14.6px
    Навигационные ссылки не должны иметь отступов, но должны быть выровнены по центру текста

Copy link

@anastasiiavorobiova anastasiiavorobiova left a comment

Choose a reason for hiding this comment

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

Great work!

- [DEMO LINK](https://<your_account>.github.io/layout_moyo-header/)
- [TEST REPORT LINK](https://<your_account>.github.io/layout_moyo-header/report/html_report/)
- [DEMO LINK](https://Teslaanazuma.github.io/layout_moyo-header/)
- [TEST REPORT LINK](https://Teslaanazuma.github.io/layout_moyo-header/report/html_report/)

Choose a reason for hiding this comment

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

Consider adding these links to the PR description

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants