Skip to content

Малахов Александр#69

Open
ghSunural wants to merge 8 commits into
urfu-2016:masterfrom
ghSunural:master
Open

Малахов Александр#69
ghSunural wants to merge 8 commits into
urfu-2016:masterfrom
ghSunural:master

Conversation

@ghSunural
Copy link
Copy Markdown

@ghSunural ghSunural commented Nov 1, 2016

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

Comment thread index.css
@font-face
{
font-family: 'CyrillicGoth';
src: url('fonts/CyrillicGoth.ttf') format('TrueType');
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Лучше еще указать атрибут local

Comment thread index.css
background-color: beige;
}

main.paperColumns
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
text-overflow: ellipsis;
}

.paperColumns 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.

В общем случае лучше использовать селектор на основе классов, а не тегов, ведь у тебя могут быть несколько тегов <p>, которые должны выглядеть по-разному. (Комментарий не только к этой строке, а ко всему коду относится)

Comment thread index.css
{
position: relative;
width: 65%;
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

.imgRoad > img
{
width: 150px;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

У картинок обязательно указывай и width и height

Comment thread index.html
умею находить умных соратников.&raquo;</cite><br>
- Ф. Д. Рузвельт<br>
<figure class="PowersThatBe">
<img src="img/PowersThatBe.jpg" 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.

Нужно указать атрибут title

Comment thread index.html
</p>
<figure class="imgRoad">
<figcaption>Дороги будущего</figcaption>
<img src="img/plasticRoad.jpg" 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.

Тут тоже title

@ninjagrizzly
Copy link
Copy Markdown

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

Comment thread index.css
html
{
background-color: aliceblue;
color: black;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

#000 короче

@ninjagrizzly
Copy link
Copy Markdown

🍅

@ninjagrizzly
Copy link
Copy Markdown

Еще в FireFox пропала полосочка
image

@ninjagrizzly
Copy link
Copy Markdown

И это место как-то странно выглядит
image

@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