Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Козлова Валерия #18

Open
wants to merge 18 commits into
base: master
Choose a base branch
from

Conversation

OnaHochetNaMars
Copy link

@OnaHochetNaMars OnaHochetNaMars commented Dec 5, 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

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

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

</head>
<body>
<main>
Copy link

Choose a reason for hiding this comment

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

❗️ <header>?

<a href="http://instagram.com">
<img class="icon" src="icons/inst.jpg" alt="instagram.com"></a>
<a href="http://telegram.me">
<img class="icon" src="icons/tg.png" alt="telegram.me"></a>
Copy link

Choose a reason for hiding this comment

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

Разметка поехала)

</form>
</div>
<div class="banner-2"></div>
<div class="left">
Copy link

Choose a reason for hiding this comment

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

❗️ А вот здесь уже main

</div>
<article class="news">
<div>
<img class="preview-photo" src="images/news1.jpg" alt="">
Copy link

Choose a reason for hiding this comment

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

❗️ Пустой alt

<div>
<div class="class"></div>
<a class="news" href="#">
В Екатеринбурге стали торговать поддельными справками о прививках от кори
Copy link

Choose a reason for hiding this comment

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

Здесь и ещё в нескольких местах не хватает отступов:

<a href="#">
    В Екатеринбурге
</a>

</article>
</section>
<section>
<div class="head">
Copy link

Choose a reason for hiding this comment

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

В принципе, страница устроена так, что можно для каждой секции сделать свой header и main, но тут уже по усмотрению)

&#9658;Перейти
</a>
</div>
<iframe src="https://www.youtube.com/2zkttxOaPr0"></iframe>
Copy link

Choose a reason for hiding this comment

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

Почему не более современный тег <video>?

left: 0;
right: 0;
top: -5px;
bottom: 05px;
Copy link

Choose a reason for hiding this comment

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

❗️ 5 или .5?


main
{
display: grid;
Copy link

Choose a reason for hiding this comment

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

В Edge grid без вендорных префиксов не работает

@Dodo888
Copy link

Dodo888 commented Dec 8, 2016

❗️ Страница шире экрана, снизу горизонатльный скролл
image

@Dodo888
Copy link

Dodo888 commented Dec 8, 2016

❗️ Главная новость должна быть ссылкой, при наведении должна появляться оранжевая рамка
image

@Dodo888
Copy link

Dodo888 commented Dec 8, 2016

❗️ При наведении заголовок новости должен становиться оранжевым
image

@Dodo888
Copy link

Dodo888 commented Dec 8, 2016

❗️ Не хватает разделителей

Твоя версия:
image

Оригинал:
image

@Dodo888
Copy link

Dodo888 commented Dec 8, 2016

❗️ Форма авторизации совсем не похожа на макет

Твоя версия:
image

Оригинал:
image

@Dodo888
Copy link

Dodo888 commented Dec 8, 2016

❗️ Неиспользуемое пространство между секциями

Твоя версия:
image

Оригинал:
image

@Dodo888
Copy link

Dodo888 commented Dec 8, 2016

❗️ Немного поехало

Твоя версия:
image

Оригинал:
image

@Dodo888
Copy link

Dodo888 commented Dec 8, 2016

❗️ Фотографии не выровнены по левому краю секции, текст не выровнен по верхнему краю фотографии

Твоя версия:
image

Оригинал:
image

@Dodo888
Copy link

Dodo888 commented Dec 8, 2016

❗️ Здесь что-то не так
image

@Dodo888
Copy link

Dodo888 commented Dec 8, 2016

❗️ Не хватает таблички

Твоя версия:
image

Оригинал:
image

@Dodo888
Copy link

Dodo888 commented Dec 8, 2016

❗️ Секции совсем непохожи на оригинал

Твоя версия:
image

Оригинал:
image

@Dodo888
Copy link

Dodo888 commented Dec 8, 2016

Я не против рыбы на странице (можно во все img положить одну и ту же картинку, в заголовки статей комбинации одного и того же текста), но хочется, чтобы все разделы были похожи на оригинальные. Пока не для всех разделов такое наблюдается (для разделов второго столбца в особенности). В плане кода очень хорошо)

@Dodo888
Copy link

Dodo888 commented Dec 8, 2016

🍅

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