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

Личный проект: завершаем вёрстку #10

Merged
merged 2 commits into from Mar 19, 2020

Conversation

@KaiZen-prog
Copy link
Contributor

KaiZen-prog commented Mar 19, 2020

@keksobot keksobot changed the title Исправление замечаний по защите Личный проект: завершаем вёрстку Mar 19, 2020
Copy link
Collaborator

efiand left a comment

Критических замечаний нет, но вот тут имхо надо бы по верхнему краю как-то сделать
изображение

Хотя может и не нарушает критерий

@@ -4,8 +4,8 @@
<meta charset="UTF-8">
<title>Каталог</title>
<link href="https://fonts.googleapis.com/css?family=Cuprum:400,700,700i%7CPT+Sans:400,700&display=swap&subset=cyrillic" rel="stylesheet">
<link href="css/normalize.css" rel="stylesheet">
<link href="css/style.css" rel="stylesheet">
<link href="css/normalize-min.css" rel="stylesheet">

This comment has been minimized.

Copy link
@efiand

efiand Mar 19, 2020

Collaborator

Чтобы подчеркнуть, что это именно минифицированная копия, обычно имя не меняют, а .min добавляют через точку: style.min.css, скрипты по тому же принципу

@@ -370,8 +370,7 @@ <h2 class="section-header">Пара слов о перфораторах</h2>
<div class="footer-lower-panel">
<div class="container footer-lower-panel-container">
<p class="copyright">
© 2010–2019 Компания «Техномарт»<br>
Все права защищены
© 2010–2019 Компания «Техномарт» Все права защищены

This comment has been minimized.

Copy link
@efiand

efiand Mar 19, 2020

Collaborator

Я бы оставил отбитие на новую строку, чтобы не так заметно )

@efiand efiand merged commit 6108ba0 into htmlacademy-htmlcss:master Mar 19, 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.