Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Личный проект: продолжаем разметку #4

Merged
merged 4 commits into from Feb 2, 2020

Conversation

@Vogrim21
Copy link
Contributor

Vogrim21 commented Feb 2, 2020

Vogrim21 added 2 commits Feb 2, 2020
Добавление попапа
Редактирование лого
@keksobot keksobot changed the title 4. Доступность и формы Личный проект: продолжаем разметку Feb 2, 2020
keksobot added a commit that referenced this pull request Feb 2, 2020
@keksobot

This comment has been minimized.

Copy link
Contributor

keksobot commented Feb 2, 2020

♻️ Я собрал ваш пулреквест. Посмотреть можно здесь.

Copy link
Collaborator

efiand left a comment

Из критичного - name в фильтрах.

catalog.html Outdated Show resolved Hide resolved
catalog.html Outdated Show resolved Hide resolved
catalog.html Outdated Show resolved Hide resolved
catalog.html Outdated Show resolved Hide resolved
catalog.html Outdated Show resolved Hide resolved
catalog.html Show resolved Hide resolved
Vogrim21 added 2 commits Feb 2, 2020
Переименование классов
исправлен label
<label for="field-5">Корзина</label>
<li class="filter-features-item">
<input type="checkbox" name="features" id="features-5">
<label for="features-5">Корзина</label>
</li>
</ul>
<button type="submit">ПОКАЗАТЬ</button>

This comment has been minimized.

Copy link
@efiand

efiand Feb 2, 2020

Collaborator

Вот тут я не заметил, и замечание не критическое, но правильнее писать по нормам языка контента (в данном случае КАПС - это крик отчаяния или аббревиатура, тут ни то ни другое. значит, пишем Показать, дабы читалка не оглушила слушателя и поисковик правильно понял важность текста, а uppercase делаем потом стилями).

<p class="products-text">Информационный сайт для туристов</p>
<a href="#nowhere" class="red">9900</a>
<p class="products-txt">Информационный сайт для туристов</p>
<a href="#nowhere" class="btn">9900</a>

This comment has been minimized.

Copy link
@efiand

efiand Feb 2, 2020

Collaborator

Еще рекомендация: вынос атрибута class вперед, так с ним удобнее работать, особенно при больших объемах кода за смену.

@efiand efiand merged commit f179357 into htmlacademy-htmlcss:master Feb 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.