search

lifelsbeautiful / web-development-course1

0 stars 0 forks source link

Замечания по lw4/task1 #4

Open ivklgn opened 3 years ago

ivklgn commented 3 years ago

.header-logo это элемент

ivklgn commented 3 years ago 
    .header-title
      .header-title_subtitle

это блок из 2 элементов:

.header-text
    .header-text__title
    .header-text__subtitle

используйте двойные подчеркивания для отделения имени элемента от блока

ivklgn commented 3 years ago

.header-notifications--disable -> .header-notification .header-notification--disable

ivklgn commented 3 years ago

.left-menu -> .sidebar .left-menu_categories -> .sidebar__menu

внутри элемент меню:

  .sidebar
     .sidebar__menu
       .sidebar__menu-item .dashboard--active (!!!)
ivklgn commented 3 years ago

.content-profile -> .profile .content-customization -> .customization

так как это отдельные блоки

ivklgn commented 3 years ago

не хватает одного блока внутри контента

.profile - есть .customization -> .accounts - есть .bills - отсутствует

ivklgn commented 3 years ago 
.customization
      .customization .my-xpay
      .customization .my-xpay_name
      .customization .my-xpay_search
      .customization .my-xpay_btn-edit
      .customization .my-xpay .first-string
      .customization .my-xpay .first-string_text
      .customization .my-xpay .first-string_btn--disable
      .customization .my-xpay .second-string
      .customization .my-xpay .second-string_text
      .customization .my-xpay .second-string_btn-active

перед .my-xpay не нужен блочный класс .customization

в .bills то же самое