Conversation
This reverts commit bffa148.
L1s-is
left a comment
There was a problem hiding this comment.
Хорошо справилась, но нужно было добавить адаптив и в некоторых местах есть вопросы по использованию тегов и стилизации
| src="./images/logo.svg" | ||
| alt="Логотип сайта" | ||
| /></a> | ||
| <h3 class="header__logo-text logo-text"> |
There was a problem hiding this comment.
Не может быть заголовка внутри логотипа, это просто текст, параграф
P.S. Если текст жирный и крупный, это не значит, что обязательно будет заголовком :)
| <meta name="msapplication-TileColor" content="#da532c" /> | ||
| <meta name="theme-color" content="#ffffff" /> | ||
| </head> | ||
| <body> |
There was a problem hiding this comment.
По общему виду:
- отсутствует адаптив
- В цветных секциях как будто бы просится, чтобы цветная часть растягивалась на всю ширину экрана пользователя https://imgur.com/nSrSGwg https://imgur.com/GkCeupy
- Прикольно, что добавила уместные анимации на карточки
| <br /> | ||
| <span class="hero__title-highlight">solutions</span> for <br />a | ||
| sustainable<br />tomorrow<span class="hero__title-highlight" | ||
| >.</span |
There was a problem hiding this comment.
Мы не должны использовать br, забудь про его существование. Перенос текста осуществляется ограничением ширины контейнера с текстом)
| display: flex; | ||
| flex-direction: column; | ||
| gap: 18px; | ||
| padding: 26px 17px 240px 23px; |
There was a problem hiding this comment.
Тут все же стоило задать min-height вместо огромного нижнего padding, поскольку если чуть увеличить размер текста одной из карточек получается фигня https://imgur.com/zrcw7D3
| <section class="expertise"> | ||
| <h2 class="expertise__title">Our Expertise and Experience</h2> | ||
| <ul class="expertise__list"> | ||
| <li class="expertise__list-item expertise__list-item_solar"> |
There was a problem hiding this comment.
- Модификатор стоит указывать через двойной дефис expertise__list-item--solar
- Именование самого элемента можно было указать как .expertise__item, оно бы тоже было уникальным(ведь БЭМ и придумали, чтобы проще и упорядоченнее было именование классов), при этом именование последующих элементов выглядело бы аккуратнее и проще .expertise__item-title итд
| </section> | ||
| <!--why-choose-us--> | ||
| <section class="why-choose-us"> | ||
| <article class="why-choose-us__heading"> |
There was a problem hiding this comment.
Я бы не сказала, что это article, поскольку article можно вынести в любое место сайта без потери смысла, тут же это невозможно
| <h2 class="why-choose-us__heading-title">Why Choose Us?</h2> | ||
| <p class="why-choose-us__heading-intro"> | ||
| Whether you're looking to switch to <br />solar energy or upgrade | ||
| your cable <br /> |
| > | ||
| <a href="#" class="who-we-are__btn who-we-are__btn--dark" | ||
| >Contact us today</a | ||
| > |
There was a problem hiding this comment.
Это скорее должна быть кнопка, а не ссылка
| } | ||
|
|
||
| .footer__links-item_contact { | ||
| display: flex; |
There was a problem hiding this comment.
Правый padding тут все же лишний, стоило убрать https://imgur.com/fwr6Bgh
The provided HTML and CSS code styles a comprehensive landing page with several distinct sections, each with its own design and purpose. Here's an overview of what has been implemented:
Header:
Hero Section:
Expertise Section:
Why Choose Us:
Who We Are:
Our Services:
Footer:
Overall, the design follows modern web design principles, using CSS flexbox and grid for layout, smooth transitions for interactivity, and specific typography and color choices to maintain a consistent brand identity across the page. This approach ensures the landing page is both visually appealing and user-friendly.