Skip to content

EAbranch#2

Open
EAliakberov wants to merge 4 commits intoalnikmakh:mainfrom
EAliakberov:EAbranch
Open

EAbranch#2
EAliakberov wants to merge 4 commits intoalnikmakh:mainfrom
EAliakberov:EAbranch

Conversation

@EAliakberov
Copy link
Copy Markdown
Contributor

Сделал разметку 1й страницы и чуток css

Copy link
Copy Markdown
Owner

@alnikmakh alnikmakh left a comment

Choose a reason for hiding this comment

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

Поправь замечания)

Comment thread css/main.css Outdated
margin: 0%;


font-family: sans-serif;
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

font-family обычно указывается у body и наследуется всеми дочерними элементами, а так тебе у каждого элемента нужно будет прописывать

Comment thread css/main.css Outdated

}

.Logo{
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

заглавные не нужны

Comment thread index.html
<img src="" alt="Картинка длинной селфи-палки">
</div>

<div class="adv-section-description">
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

если это БЭМ, то тут что-то не так с правилами нейминга) элемент через двойное нижнее подчеркивание, модификатор через двойное тире. При этом нельзя просто класс-модификатор применить к тегу. Модификатор на то и модификатор, чтобы работать в дополнении основного класса

Comment thread index.html
<section class="last-section">
<h1 class="hidden-header">Последняя секция</h1>
<section class="section-about-us">
<h1 class="header1">О нас</h1>
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

учись сразу называть нормально)

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