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

Смирнов Алексей #36

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Смирнов Алексей #36

wants to merge 8 commits into from

Conversation

AlexSm13
Copy link

@AlexSm13 AlexSm13 commented Oct 30, 2016

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍏 Пройден линтинг и базовые тесты

@FrodoTheTrue
Copy link

В целом ок. Хочется еще увидеть:

  1. Картиночек среди колонок
  2. Вертикальный текст
  3. Будет круто если подключишь шрифты не только с google fonts

@FrodoTheTrue
Copy link

🍅

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍏 Пройден линтинг и базовые тесты

column-rule: 1px solid rgba(0,0,0, .5);
-moz-column-rule: 1px solid rgba(0,0,0, .5);
-webkit-column-rule: 1px solid rgba(0,0,0, .5);
hyphens: auto;
break-inside: avoid-page;
}

.post-2
{
column-count: 3;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

безвендорное правило должно идти после

@FrodoTheTrue
Copy link

Стало лучше, хочется еще увидеть картинку на несколько стобцов

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍏 Пройден линтинг и базовые тесты

@FrodoTheTrue
Copy link

FrodoTheTrue commented Nov 2, 2016

В мозиле совсем разломалось
2016-11-02 23 03 16
В этом браузере есть некоторые проблемы, но верстка не должна так сильно разваливаться и текст должен быть читаемым

<header>
<h1 class="headline">USA TODAY</h1>
<aside class="weather"><span class="vertical">Weather</span>
<img src="image/d.sun.png" alt="sun">+12</aside>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

не хватает еще title

@FrodoTheTrue
Copy link

🍅

height: 140px;
}

div

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

привязываться к тегам обычно плохая практика, а к диву особенно нехорошо

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

Successfully merging this pull request may close these issues.

3 participants