Skip to content

Набойченко Георгий#95

Open
georgy-n wants to merge 3 commits intourfu-2018:masterfrom
georgy-n:master
Open

Набойченко Георгий#95
georgy-n wants to merge 3 commits intourfu-2018:masterfrom
georgy-n:master

Conversation

@georgy-n
Copy link
Copy Markdown

@georgy-n georgy-n commented Oct 23, 2018

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

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

main
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.css
border-bottom: 1px double black;
}

.article3 .content
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

можно объединить с .article2 .content, а еще лучше не называть классы 1,2,3 а делать более понятными

Comment thread index.html
<p class="paper-type">Index out of range</p>
</section>
<main>
<article class="article1">
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
<figure>
<img
src="https://avatars.mds.yandex.net/get-pdb/27625/abe819b7-5ba8-4202-87fd-108323e7e26d/s1200"
alt="big bear">
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 
                    src="https://avatars.mds.yandex.net/get-pdb/27625/abe819b7-5ba8-4202-87fd-108323e7e26d/s1200"
                    alt="big bear"
                >

лучше так выравнять тогда уж

Comment thread index.html
</figure>
</article>
<article class="article2">
<h1 class="article-title">Дом снесут ради здания филармонии</h1>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

кажется, тут должны быть уже h2, ведь h1 был выше

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants