Skip to content

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

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

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

Conversation

@NeSmogPridymatNick

@NeSmogPridymatNick NeSmogPridymatNick commented Oct 31, 2016 •

Copy link
Copy Markdown

@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

forshtreter commented Nov 11, 2016 •

Copy link
Copy Markdown

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