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

Личный проект: начинаем готовить графику #3

Merged
merged 1 commit into from Feb 2, 2020

Conversation

@KaiZen-prog
Copy link
Contributor

KaiZen-prog commented Feb 2, 2020

@keksobot keksobot changed the title Подключение контентных изображений к разметке Личный проект: начинаем готовить графику Feb 2, 2020
Copy link
Collaborator

efiand left a comment

Задание принимаю, по типам файлов и разметке нарушений нет, имена можно поправить в следующем задании

@@ -62,61 +60,61 @@ <h3>Форма фильтрации товаров</h3>
<ul>
<li>
<h3>Перфоратор BOSCH BFG 3000</h3>
<img src="https://via.placeholder.com/100x150" alt="Перфоратор BOSCH BFG 3000">
<img src="img/Perforator_1.jpg" width="218" height="170" alt="Перфоратор BOSCH BFG 3000">

This comment has been minimized.

Copy link
@efiand

efiand Feb 2, 2020

Collaborator

С точки зрения кроссплатформенности: лучше избегать больших букв в названиях файлов, к тому же чаще всего в качестве разделителей приняты дефисы.
Поскольку дефис у нас уже есть для лого, тут мы нарушаем критерий единообразия.

@efiand efiand merged commit 4f203c3 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

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