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 2 commits into from Feb 4, 2020

Conversation

@KaiZen-prog
Copy link
Contributor

KaiZen-prog commented Feb 2, 2020

@keksobot keksobot changed the title Продолжение разметки Личный проект: дополняем разметку Feb 2, 2020
catalog.html Outdated Show resolved Hide resolved
catalog.html Show resolved Hide resolved
catalog.html Outdated Show resolved Hide resolved
catalog.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
Copy link
Collaborator

efiand left a comment

Оставшиеся замечания, чтобы не тянуть время, просьба учесть при доработке доступной верстки.

<a href="#nowhere">Для стройки</a>
</li>
</ul>

<div>

This comment has been minimized.

Copy link
@efiand

efiand Feb 4, 2020

Collaborator

Слайдер невозможно сделать, не обернув все слайды в контейнер (либо дивы в див, либо сделав список - для слайдера он уместен).

</ul>
<p>Дрели</p>
<p>Соседям на радость!</p>
<a href="#nowhere">Назад</a>

This comment has been minimized.

Copy link
@efiand

efiand Feb 4, 2020

Collaborator

В отличие от ссылки на каталог (где будет открываться соотв. раздел каталога), здесь Назад и Вперед - по одной на весь слайдер, и это button, ибо промо-слайдер никакого критически важного контента не несет. при отсутствии js просто будет отрисован первый слайд и навигация слайдера скрыта.

@efiand efiand merged commit 20f11af into htmlacademy-htmlcss:master Feb 4, 2020
@keksobot

This comment has been minimized.

Copy link
Contributor

keksobot commented Feb 4, 2020

Не удалось смёржить пулреквест. Проверьте наличие конфликтов. Задание переведено в статус Есть замечания

2 similar comments
@keksobot

This comment has been minimized.

Copy link
Contributor

keksobot commented Feb 4, 2020

Не удалось смёржить пулреквест. Проверьте наличие конфликтов. Задание переведено в статус Есть замечания

@keksobot

This comment has been minimized.

Copy link
Contributor

keksobot commented Feb 4, 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.