nevermin9 / authentic Goto Github PK

Те ж саме, що і в #33
Перевірте, будь ласка, це явище в усіх файлах :)

https://github.com/Platoniux/Authentic/blob/33017adadbbdcf7b5d2aed5eeb19a74f3847a0ee/src/scss/_start.scss#L52-L115

https://github.com/Platoniux/Authentic/blob/33017adadbbdcf7b5d2aed5eeb19a74f3847a0ee/src/scss/_timeline.scss#L180

В селекторі '& img {' не потрібно писати &. Це буде те ж саме.

.brands {
    img {

А ще, тут для тега img теж варто створити клас :) Наприклад, а що якщо ви вирішите вставляти картинки в <svg> :)

https://github.com/Platoniux/Authentic/blob/33017adadbbdcf7b5d2aed5eeb19a74f3847a0ee/src/scss/_brands.scss#L103

Клас модифікатора без класу елемента
Але і для цього класу модифікатора немає стилів, тому, він там взагалі не потрібен :)

https://github.com/Platoniux/Authentic/blob/33017adadbbdcf7b5d2aed5eeb19a74f3847a0ee/src/index.html#L596

https://github.com/Platoniux/Authentic/blob/33017adadbbdcf7b5d2aed5eeb19a74f3847a0ee/src/scss/_social.scss#L54

Зона дії hover для our-member__cont значно більша за розмір блока
Якщо під картинкою навести мишкою - спрацьовує hover. Мабуть, щось трішки не те..

Не вистачає ще однієї закриваючої дужки } вкінці _main-header.scss
https://github.com/Platoniux/Authentic/blob/33017adadbbdcf7b5d2aed5eeb19a74f3847a0ee/src/scss/_main-header.scss#L259

https://github.com/Platoniux/Authentic/blob/33017adadbbdcf7b5d2aed5eeb19a74f3847a0ee/src/scss/_videocontainer.scss#L54-L73

&__item {
  &-link {

Так, на жаль, писати не можна :(
main-header__item-link - це один елемент. А main-header__item - це інший.
Вкладеність можна використовувати для повної назви елементу або для модифікаторів.
Те, що в нас класи елементів починають з одного слова - це просто співпадіння і мають бути незалежні правила для
&__item i &__item-link

https://github.com/Platoniux/Authentic/blob/33017adadbbdcf7b5d2aed5eeb19a74f3847a0ee/src/scss/_main-header.scss#L251

Якщо збільшити заголовок book-a-seat__title до декількох стрічок, він виглядає не дуже гарно

hover потрібно прописувати для елементів, на які відбуватиметься клік. Тобто на <a>, а не на <li>.

Ці два правила

     &:hover {
        background: $main-color;
      }

      &:hover .main-header__item-link {
        color: $text-start;
      }

Можна було об'єднати в одне:

    &:hover {
      background: $main-color;

      .main-header__item-link {
        color: $text-start;
      }
    }

https://github.com/Platoniux/Authentic/blob/33017adadbbdcf7b5d2aed5eeb19a74f3847a0ee/src/scss/_main-header.scss#L239-L243

Якщо в our-blog збільшити заголовок - лінія під загловком виглядає не так, як потрібно. І блоки не вирівняні по верхній лінії картинки.

Також варто додати посилання, щоб була можливість перейти в конкретну статтю блогу.

Відсутня ; вкінці стрічки :)
і ще в 76 стрічці

https://github.com/Platoniux/Authentic/blob/33017adadbbdcf7b5d2aed5eeb19a74f3847a0ee/src/scss/_brands.scss#L87

Для textarea краще заборонити зміну розміру, принаймні, в ширину. Це може сильно поломати верстку всього сайту :)

https://github.com/Platoniux/Authentic/blob/33017adadbbdcf7b5d2aed5eeb19a74f3847a0ee/src/index.html#L465

Тут, на перший погляд, здається, що занадто багато брейкпоінтів. Можливо, можна якось продумати універсальніше? в якихось відносних одиницях (хоча там і так одиниці відносні до вьювпорта)...Подумайте, як це можна змінити :)
та й взагалі, чекбокс невидимий.. для чого йому аж так задавати значення right?

І з брейкпоінтами для &__burger-line теж пропоную щось придумати :)

https://github.com/Platoniux/Authentic/blob/33017adadbbdcf7b5d2aed5eeb19a74f3847a0ee/src/scss/_main-header.scss#L47-L103

А ще в цому ж файлі в 94 стрічці - те ж саме, що і в #33

https://github.com/Platoniux/Authentic/blob/33017adadbbdcf7b5d2aed5eeb19a74f3847a0ee/src/scss/_additional.scss#L143-L146

При ховері на our-member__cont відсутній transition - дуже різко з'являється сірий фон.

Те ж саме, що і #33

https://github.com/Platoniux/Authentic/blob/33017adadbbdcf7b5d2aed5eeb19a74f3847a0ee/src/scss/_timeline.scss#L119-L129