search

DanilLebedkin / Internet_Programming

0 stars 0 forks source link

Review LR4 #4

Open steminist-ua opened 1 year ago

steminist-ua commented 1 year ago

не коректно задано жирне накреслення для елементів списку.

  1. елемент b мав би знаходитися у li
  2. це виділення стосується виключно стилю, а не контенту, тому коректно це робити саме через стилі, а не html

https://github.com/DanilLebedkin/Internet_Programming/blob/6ea459149cabdc7e3821bfc8e7e31f1f6915990d/%D0%9B%D0%A04/LR4.html#L25-L31

steminist-ua commented 1 year ago
  1. меню чомусь за структурою розірвано, хоча це одне меню сторінки. Воно має включати порожній елемент з блоком, який потрібно зробити колом у стилях
  2. текст у меню капсом робиться через стилі, а не у коді набирається самостійно

https://github.com/DanilLebedkin/Internet_Programming/blob/6ea459149cabdc7e3821bfc8e7e31f1f6915990d/%D0%9B%D0%A04/LR4.html#L25-L35

steminist-ua commented 1 year ago

не коректне підключення шрифтів

https://github.com/DanilLebedkin/Internet_Programming/blob/6ea459149cabdc7e3821bfc8e7e31f1f6915990d/%D0%9B%D0%A04/style.scss#L10-L27

steminist-ua commented 1 year ago

через не коректну структуру сторінки, то й стиль меню налаштовано не коректно

https://github.com/DanilLebedkin/Internet_Programming/blob/6ea459149cabdc7e3821bfc8e7e31f1f6915990d/%D0%9B%D0%A04/style.scss#L119-L135

steminist-ua commented 1 year ago

не коректне використання rem, оскільки не змінюється розмір шрифту у корінні документу (в селекторі html у мобільній адаптації). При такому використанні немає сенсу його задіювати, оскільки все одно доводиться для всіх елементів переписувати, змінювати значення

у стилях за замовчуванням задано https://github.com/DanilLebedkin/Internet_Programming/blob/6ea459149cabdc7e3821bfc8e7e31f1f6915990d/%D0%9B%D0%A04/style.scss#L6-L8

у стилях для адаптації - немає https://github.com/DanilLebedkin/Internet_Programming/blob/6ea459149cabdc7e3821bfc8e7e31f1f6915990d/%D0%9B%D0%A04/style.scss#L137-L141