Skip to content

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

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

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

Conversation

@NeSmogPridymatNick
Copy link
Copy Markdown

@NeSmogPridymatNick NeSmogPridymatNick commented Oct 31, 2016

@honest-hrundel honest-hrundel changed the title Хмелев Дмитрий Хмелёв Дима Oct 31, 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

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

@onufrienko
Copy link
Copy Markdown

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

Comment thread index.css Outdated
border: 5px double;
}

.headerOne
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

first-header

Comment thread index.css Outdated
height: 75%;
}

.headerTwo
Copy link
Copy Markdown

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
Copy Markdown

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

Comment thread index.css Outdated
width: 15%;
}

.headerName
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 Outdated
font-family: 'Lobster', cursive;
}

.first
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 Outdated
width: 40%;
}

.second,
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

@honest-hrundel
Copy link
Copy Markdown

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

Comment thread index.css Outdated
.blackRect
{
background-color: black;
width: 20px !important;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Comment thread index.css Outdated
background-color: black;
}

.see
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 Outdated
width: 100%;
}

.ogl
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

ogl? :)

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

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

Comment thread index.css Outdated
width: 300px;
height: 216px;
border: 2px double;
float: right;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

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

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 Outdated

.page
{
float: right;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

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 Outdated
font-size: 10px;
}

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

Choose a reason for hiding this comment

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

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

Comment thread index.html Outdated
под одной крышей
</div>
</figure>
</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.

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

@onufrienko
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.html Outdated
Факторов, определивших успех Прибамбасска, мэр
назвал удобное прибрежное расположение города —
рядом с Великим морем, а так же политику всячиски
рядом с Великим морем, а так же политику всячиски
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

всячески

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

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

@onufrienko
Copy link
Copy Markdown

🍅

@honest-hrundel
Copy link
Copy Markdown

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

@onufrienko
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@onufrienko
Copy link
Copy Markdown

🚀

@forshtreter
Copy link
Copy Markdown

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

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

Comment thread index.html
выпуск
</div>
<div class="header-name">
Вестник Прибамбасска<br>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Зачем этот br?

Comment thread index.html
<img src="img/Море.png" alt="Море" title="Море">
</p>
<main>
<table>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

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

Comment thread index.html
<h1>
Доходы растут
</h1>
<p class="main-picture">
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 – это paragraph, использовать его в качестве обёртки для картинки не лучшая идея, существует более специализированный тег именно для этой цели

Comment thread index.html
<img src="img/Тотем.png" class="totem" alt="ТотемыРеклама" title="Реклама">
</td>
<td>
<div class="table-of-contents">
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
.header
{
position: relative;
overflow: hidden;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Зачем overflow: hidden?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

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

Comment thread index.css

.first-header
{
height: 75%;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

75% чего?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

75% всего header

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

@forshtreter
Copy link
Copy Markdown

forshtreter commented Nov 11, 2016

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

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

@forshtreter
Copy link
Copy Markdown

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

@NeSmogPridymatNick
Copy link
Copy Markdown
Author

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

@forshtreter
Copy link
Copy Markdown

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

@forshtreter
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.

4 participants