Skip to content

Трофимов Павел#55

Open
tgkd wants to merge 2 commits intourfu-2016:masterfrom
tgkd:master
Open

Трофимов Павел#55
tgkd wants to merge 2 commits intourfu-2016:masterfrom
tgkd:master

Conversation

@tgkd
Copy link
Copy Markdown

@tgkd tgkd commented Oct 31, 2016

@honest-hrundel
Copy link
Copy Markdown

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

Comment thread index.css
.container
{
border: 2px double black;
font-size: 18px;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Используя свойство font получится уложить 2 следующие строчки в одну

Comment thread index.css
text-align: right;
}

article
Copy link
Copy Markdown

@ninjagrizzly ninjagrizzly Nov 2, 2016

Choose a reason for hiding this comment

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

Лучше использовать селекторы по классам, возможна ситуация, когда на странице есть несколько элементов <article>, которые должны выглядеть по-разному (комментарий относится ко всему коду, а не только к данному селектору)

Comment thread index.css

article em
{
column-span: all;
Copy link
Copy Markdown

@ninjagrizzly ninjagrizzly Nov 2, 2016

Choose a reason for hiding this comment

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

Обрати внимание, это свойство у тебя не срабатывает для <em>.

Comment thread index.css
.second-column,
.third-column
{
float: left;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Свойство float в данной задаче нельзя использовать для разметки страницы

Comment thread index.css

.earth-img
{
width: 95%;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Высоту картинки тоже нужно указывать

Comment thread index.html
<body>
<div class="container">
<header>
<div class="page-name">Ведомости</div>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Для заголовков есть специальные теги <h1> - <h6>

Comment thread index.html
<div class="page-name">Ведомости</div>
</header>
<hr>
<p class="page-info">Выпуск от 30.10.2016 №10</p>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Дату можно обернуть в специальный тег

Comment thread index.html
<hr>
<p class="page-info">Выпуск от 30.10.2016 №10</p>
<hr>
<div class="page-body">
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Тег <main>

Comment thread index.html
<hr>
<div class="page-body">
<div class="first-column">
<div class="section">Интересное</div>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Это тоже заголовок

Comment thread index.html
</article>
</div>
<div class="second-column">
<img class="earth-img" src="https://goo.gl/OuaQf3" alt="Земля">
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

У тега <img> обязательно указывать непустые атрибуты alt и title

Comment thread index.html
</article>
</div>
<div class="third-column">
<div class="section">Новости спорта</div>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

И это заголовок

@ninjagrizzly
Copy link
Copy Markdown

Не соответствует требованиям задачи:

  1. Не нашел вертикального текста
  2. Никак не выделяется первая буква первого параграфа статьи
  3. Нужно использовать нестандартные шрифты, подключив их в css файле

@ninjagrizzly
Copy link
Copy Markdown

🍅

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