Skip to content

Хмелёв Дима#51

Open
NeSmogPridymatNick wants to merge 16 commits intourfu-2016:masterfrom
NeSmogPridymatNick:master
Open

Хмелёв Дима#51
NeSmogPridymatNick wants to merge 16 commits intourfu-2016:masterfrom
NeSmogPridymatNick:master

Conversation

@NeSmogPridymatNick
Copy link

@NeSmogPridymatNick NeSmogPridymatNick commented Oct 31, 2016

@honest-hrundel honest-hrundel changed the title Хмелев Дмитрий Хмелёв Дима Oct 31, 2016
@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@onufrienko
Copy link

Куда-то делись картинки :(
image

index.css Outdated
border: 5px double;
}

.headerOne

Choose a reason for hiding this comment

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

first-header

index.css Outdated
height: 75%;
}

.headerTwo

Choose a reason for hiding this comment

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

second-header

@onufrienko
Copy link

Не используй camelCase для именования классов

index.css Outdated
width: 15%;
}

.headerName

Choose a reason for hiding this comment

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

Не очень удачное имя класса

index.css Outdated
font-family: 'Lobster', cursive;
}

.first

Choose a reason for hiding this comment

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

Первое что?

index.css Outdated
width: 40%;
}

.second,

Choose a reason for hiding this comment

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

Второе, третье, четвертое что? :)

@honest-hrundel
Copy link

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

index.css Outdated
.blackRect
{
background-color: black;
width: 20px !important;

Choose a reason for hiding this comment

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

Попробуй обойтись без !important

index.css Outdated
background-color: black;
}

.see

Choose a reason for hiding this comment

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

Опять-таки не очень понятное имя

index.css Outdated
width: 100%;
}

.ogl

Choose a reason for hiding this comment

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

ogl? :)

Copy link
Author

Choose a reason for hiding this comment

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

черновой вариант , забыл исправить. Классов чет многа

index.css Outdated
width: 300px;
height: 216px;
border: 2px double;
float: right;

Choose a reason for hiding this comment

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

В этом задании нужно обойтись без плавающих элементов

Copy link
Author

Choose a reason for hiding this comment

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

А как тогда выравнивать картинки относительно блоков? Или вставлять картинку в середину текста?

Choose a reason for hiding this comment

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

Творить магию :)

index.css Outdated

.page
{
float: right;

Choose a reason for hiding this comment

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

То же замечание, что и выше

Choose a reason for hiding this comment

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

Либо что-то пошло не так, либо я все еще вижу float

index.css Outdated
font-size: 10px;
}

.pictureMainFirst

Choose a reason for hiding this comment

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

поясни, пожалуйста, название класса?

<title>Прибамбасск</title>
<meta charset="UTF-8">
<link href="index.css" rel="stylesheet">
<link href="https://fonts.googleapis.com/css?family=Lobster" rel="stylesheet">

Choose a reason for hiding this comment

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

Будет хорошо, если попробуешь подключить шрифты не только с google fonts, но и самостоятельно

index.html Outdated
под одной крышей
</div>
</figure>
</div>

Choose a reason for hiding this comment

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

почему ты переделал на div?

@onufrienko
Copy link

🍅

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

index.html Outdated
Факторов, определивших успех Прибамбасска, мэр
назвал удобное прибрежное расположение города —
рядом с Великим морем, а так же политику всячиски
рядом с Великим морем, а так же политику всячиски

Choose a reason for hiding this comment

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

всячески

Copy link
Author

Choose a reason for hiding this comment

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

ахахахаха , я с картинки программой текст получал)

@onufrienko
Copy link

🍅

@honest-hrundel
Copy link

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

@onufrienko
Copy link

Почему я до сих пор вижу плавающий элемент и font-size small? :(

@honest-hrundel
Copy link

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

@onufrienko
Copy link

🚀

@forshtreter
Copy link

Тематически очень хорошо, реализация пока что не особо.

  • Шапка растягивается, содержимое – нет, выглядит странно. https://yadi.sk/i/F3WkZb47yQ9JC Надо растягивать содержимое тоже, тогда начинается самое интересное
  • Оглавление развалилось https://yadi.sk/i/jH9mh75zyQ9ay
  • Между баннерами и текстом какие-то странные дыры, между собой баннеры тоже как-то странно выровнены https://yadi.sk/i/aDNiCBXRyQ9ki

выпуск
</div>
<div class="header-name">
Вестник Прибамбасска<br>

Choose a reason for hiding this comment

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

Зачем этот br?

<img src="img/Море.png" alt="Море" title="Море">
</p>
<main>
<table>

Choose a reason for hiding this comment

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

Таблица тут вобще для раскладки не нужна.

Copy link
Author

Choose a reason for hiding this comment

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

ну тогда сложно padding-ми все выравнивать , так как float то же нельзя.Я первый раз все картинки флоатами выравнивал. Сказали нельзя. Я могу все inline дивами , все сделать. Но внутри все равно придется паддингами двигать

<h1>
Доходы растут
</h1>
<p class="main-picture">

Choose a reason for hiding this comment

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

p – это paragraph, использовать его в качестве обёртки для картинки не лучшая идея, существует более специализированный тег именно для этой цели

<img src="img/Тотем.png" class="totem" alt="ТотемыРеклама" title="Реклама">
</td>
<td>
<div class="table-of-contents">

Choose a reason for hiding this comment

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

Оглавление можно сверстать как-нибудь поизящней, списком например

.header
{
position: relative;
overflow: hidden;

Choose a reason for hiding this comment

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

Зачем overflow: hidden?

Copy link
Author

Choose a reason for hiding this comment

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

черновой вариант , забыл удалить


.first-header
{
height: 75%;

Choose a reason for hiding this comment

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

75% чего?

Copy link
Author

Choose a reason for hiding this comment

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

75% всего header

Choose a reason for hiding this comment

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

Дело в том что у header-то высота не задана и поэтому это правило не работает.

@forshtreter
Copy link

forshtreter commented Nov 11, 2016

https://yadi.sk/i/eY2uheF4yQCxP этот блок такой странный, что я не могу понять, он так и должен выглядеть или развалился весь :)

Не знаю даже что посоветовать, давайте попробуем сделать хотя бы так, чтобы элементы в нём как-то более равномерно пространство занимали чтоли. Можно ещё купон выделить с помошью dashed бордера

@forshtreter
Copy link

То что надпись "ежедневно" прыгает туда-сюда в зависимости от ширины экрана это баг или фича? https://yadi.sk/i/Jk4snAvHyQDr4

@NeSmogPridymatNick
Copy link
Author

"этот блок такой странный, что я не могу понять, он так и должен выглядеть или развалился весь"
Да , так и должен выглядеть
http://natpagle.ru/wp-content/uploads/2016/10/vestnik-pribambasska-gazeta-02-0.jpg

@forshtreter
Copy link

Окей, этот вопрос снимается.

@forshtreter
Copy link

Но остальное в силе. Забыл помидор. 🍅

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.

4 participants

Comments