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

Lesson 5 #1

Open
wants to merge 3 commits into
base: Lesson-5
Choose a base branch
from
Open

Lesson 5 #1

wants to merge 3 commits into from

Conversation

Enterigo
Copy link

Значення letter-spacing в фш = 0!

@inlen inlen changed the title Lesson-5 Lesson 5 Oct 12, 2017
@inlen
Copy link
Owner

inlen commented Oct 12, 2017

  1. Треба підлючити нормалайз
  2. Треба поставити правильний порядок підключення css спочатку шрифт, потім нормалайз аж потім style
  3. Весь контент встав в артікл, це ж стаття і інформацію про ню, просто верхню частину можна виділити як header
  4. Чому використав не h1, a h2?
  5. Правильно буде перерахувати line-height в коефіцієнт
  6. Для того щоб не писати для кожного селектора сімейство шрифтів можна основний обявити як базовий, тобто дати для html, або article якщо відноситься до стилів лише статті)

@inlen inlen added the wontfix label Oct 12, 2017
@Enterigo
Copy link
Author

Enterigo commented Oct 12, 2017 via email

@inlen
Copy link
Owner

inlen commented Oct 12, 2017

Очікую правки)

@Enterigo
Copy link
Author

Enterigo commented Oct 12, 2017 via email

@Enterigo
Copy link
Author

Enterigo commented Nov 10, 2017

Відібрав ноут у друга, за вихідні все маю зробити xD

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants