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

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

Merged
merged 7 commits into from Feb 10, 2020

Conversation

@KaiZen-prog
Copy link
Contributor

KaiZen-prog commented Feb 7, 2020

Экспортированы слайды для главной страницы и флаг "new" для каталога


🎓 Личный проект: продолжаем подготовку графики


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

Экспортированы слайды для главной страницы и флаг "new" для каталога
@keksobot keksobot changed the title Продолжение подготовки графики Личный проект: продолжаем подготовку графики Feb 7, 2020
@keksobot keksobot changed the title Личный проект: продолжаем подготовку графики Личный проект: продолжаем разметку Feb 9, 2020
img/new-flag.svg Outdated Show resolved Hide resolved
Copy link
Collaborator

efiand left a comment

Сейчас мы видим принципиально неверный подход: если вы не поняли, как сохранить вектор, не стоит его подменять растром, растр подходит только на худой конец, когда вектора нам не завезли вообще. Если надо, можно сделать созвон по экспорту изображений.
Кстати говоря, new рекомендую делать контентом (параграф или за счет псевдоэлемента потом, а пока добавив класс), картинка svg, если она не содержит живого текста, читалками понята не будет.
Если вектора new в макете нет, делаем не через графику. А то это фактически тот же подход, что Вадим показывало в начале 4 лекции ))

Очевидно, что при узнавании о проблеме стоит исправить все подобные ситуации, сейчас такая же проблема и с фонами второго слайдера. Не припомню, чтобы там был svg в смарт-объекте, вот тут png скорее всего в чистом виде.

Еще рекомендация так не именовать иконки
изображение

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

catalog.html 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 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
@efiand

This comment has been minimized.

Copy link
Collaborator

efiand commented Feb 9, 2020

изображение

Рекомендация: в монохромных иконках вешать fill непосредственно на svg, это позволит единообразно стилизовать, если иконка в разметке. А единственную группу вообще стоит удалять, она никакой пользы не приносит, лучше конечно дизайнером при экспорте, ну раз так вышло, то вручную верстальщиком: минификатору тут доверять не стоит.

@efiand

This comment has been minimized.

Copy link
Collaborator

efiand commented Feb 10, 2020

Рекомендация на дальнейшие задания:
попробовать реализовать это с помощью css, нужная помощь будет оказана
изображение
В случае реализации иконки удалить.

Copy link
Collaborator

efiand left a comment

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

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
@efiand efiand merged commit 148c91f into htmlacademy-htmlcss:master Feb 10, 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.