Сергеева Наталья by Pogromistka · Pull Request #60 · urfu-2016/markup-task-4

Pogromistka · 2016-10-31T18:48:53Z

Посмотреть решение

honest-hrundel · 2016-10-31T18:49:37Z

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

honest-hrundel · 2016-10-31T18:51:12Z

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

honest-hrundel · 2016-10-31T19:01:46Z

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

honest-hrundel · 2016-10-31T19:06:08Z

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

honest-hrundel · 2016-10-31T19:15:00Z

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

honest-hrundel · 2016-11-01T05:00:47Z

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

honest-hrundel · 2016-11-01T05:11:41Z

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

honest-hrundel · 2016-11-01T05:14:26Z

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

VasiliiKuznecov · 2016-11-02T16:47:21Z

Во второй колонке так мало текста, что появляется вопрос, а зачем вообще вторая колонка?)
Смотрится странно)

VasiliiKuznecov · 2016-11-02T16:51:10Z

index.css

+p,
+td
+{
+    font-family: Century Gothic, Bookman Old Style;


лучше задавать не каждому элементу, а родителю, который содержит эти элементы

VasiliiKuznecov · 2016-11-02T16:51:31Z

index.css

+    font-family: Century Gothic, Bookman Old Style;
+}
+
+.tit p


не сокращай названия классов

VasiliiKuznecov · 2016-11-02T16:52:35Z

index.html

 </head>
 <body>
+    <header>
+        <div class="tit"><p>Slamdunk</p></div>


Зачем внутри p? И без него можно обойтись

VasiliiKuznecov · 2016-11-02T16:54:17Z

index.html

 <body>
+    <header>
+        <div class="tit"><p>Slamdunk</p></div>
+        <div class="dat"><p>ПОНЕДЕЛЬНИК, ОКТЯБРЬ 31, 2016</p></div>


то же самое + не пиши текст капсом, а используй text-transform: uppercase

VasiliiKuznecov · 2016-11-02T16:57:56Z

index.css

+{
+    font-family: Bodoni MT Black, Bookman Old Style;
+    font-style: italic;
+    font-size: 500%;


Лучше размер шрифта либо указать в пикселях, либо в em при условии, что у родительского элемента указали font-size

VasiliiKuznecov · 2016-11-02T16:58:29Z

index.css

+    font-style: italic;
+    font-size: 500%;
+    text-align: center;
+    margin-top: 0;


без p не пришлось бы этого делать)

VasiliiKuznecov · 2016-11-02T16:59:01Z

index.css

+.dat
+{
+    font-family: Century Gothic, Bookman Old Style;
+    font-style: normal;


по умолчанию normal

VasiliiKuznecov · 2016-11-02T17:00:35Z

index.css

+    border-bottom-style: solid;
+}
+
+img


Лучше не использовать в селекторах теги, а использовать классы

VasiliiKuznecov · 2016-11-02T17:02:23Z

index.css

+    padding-top: 10px;
+}
+
+.allnews


почему этот класс называется allnews? Там не все новости)

VasiliiKuznecov · 2016-11-02T17:04:48Z

В firefox очень странно выглядит последняя строчка новости в левой колонке(

honest-hrundel · 2016-11-09T04:30:43Z

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

honest-hrundel · 2016-11-09T05:52:00Z

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

honest-hrundel · 2016-11-09T05:57:59Z

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

для таблиц разные классы создала .results .schedule

honest-hrundel · 2016-11-09T06:06:52Z

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

honest-hrundel · 2016-11-09T06:08:07Z

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

honest-hrundel · 2016-11-09T06:12:47Z

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

honest-hrundel · 2016-11-09T06:21:50Z

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

honest-hrundel · 2016-11-09T08:30:47Z

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

honest-hrundel · 2016-11-09T08:32:10Z

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

msmirnov · 2016-11-09T08:58:36Z

При ширине 800px левые границы прилипают к браузеру, читать неудобно. Нужны какие-то отступы.

msmirnov · 2016-11-09T09:00:18Z

index.css

+{
+    font-family: 'Bodoni MT Black', 'Bookman Old Style';
+    font-style: italic;
+    font-size: 5em;


Вот эти три строчки можно свернуть в
font: italic 5em 'Bodoni MT Black', 'Bookman Old Style';

Также всем шрифтам следует указать дефолтное значение на случай, если шрифты не загрузятся (напр, serif, sans-serif)

msmirnov · 2016-11-09T09:00:52Z

index.css

+.date
+{
+    font-family: 'Century Gothic', 'Bookman Old Style';
+    font-size: 12px;


Также нужно свернуть два свойства в одно

msmirnov · 2016-11-09T09:01:57Z

🍅

msmirnov · 2016-11-09T09:47:58Z

Кстати, у меня до сих пор не появились шрифты.

Похоже, ты их указала, но нигде не подключила

honest-hrundel · 2016-11-09T19:18:01Z

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

honest-hrundel · 2016-11-09T19:19:10Z

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

msmirnov · 2016-11-09T21:22:52Z

Ты зачем-то подключила 'Century Gothic' с начертанием italic и нигде его не использовала

msmirnov · 2016-11-09T21:25:18Z

Что такое Bookman Old Style? Это нестандартный шрифт, а в списке подключенных я его не вижу. К тому же, при определении шрифтов, все еще не указаны шрифты по умолчанию. Можно вот здесь почитать: http://htmlbook.ru/css/font

msmirnov · 2016-11-09T21:26:06Z

index.css

 {
-    font-family: 'Bodoni MT Black', 'Bookman Old Style';
-    font-style: italic;
+    font: italic 'Bodoni MT Black', 'Bookman Old Style';


Ты проверила и это работает?

Я вижу это как обычный Times:

msmirnov · 2016-11-09T21:28:05Z

🍅

Pogromistka added 3 commits October 31, 2016 23:45

хтмл

7a1063f

ксс

4126e87

Add files via upload

aa26cde

хтмл

67f570b

ксс ошибки исправлены

da8cac8

ксс исправление 2

dc6fc7e

хтмл ошибки

1004646

хтмл длина строк

e36bbdf

и ещё раз хтмл

439c552

финальный вариант

e040064

honest-hrundel assigned VasiliiKuznecov Nov 1, 2016

VasiliiKuznecov reviewed Nov 2, 2016

View reviewed changes

index.css Outdated

font-family: Century Gothic, Bookman Old Style;

}

.tit p

Copy link

VasiliiKuznecov Nov 2, 2016

Choose a reason for hiding this comment

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

не сокращай названия классов

VasiliiKuznecov reviewed Nov 2, 2016

View reviewed changes

index.css Outdated

.dat

{

font-family: Century Gothic, Bookman Old Style;

font-style: normal;

Copy link

VasiliiKuznecov Nov 2, 2016

Choose a reason for hiding this comment

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

по умолчанию normal

VasiliiKuznecov reviewed Nov 2, 2016

View reviewed changes

Кавычки css

b05e9cf

Переносы и подпись картинки

87cd70c

css подпись картинки и выделение букв

f719f2c

css подпись картинки и выделение букв (2)

822c5af

для таблиц разные классы создала .results .schedule

html классы таблиц .results .schedule

6c2ed8b

css таблица результатов скачет

2d2c7d1

css выровняла табличку

b9996df

css вертикал текст

2b21c74

html вертикал текст

c10749a

msmirnov reviewed Nov 9, 2016

View reviewed changes

css шрифты и отступы

b764811

Шрифты

2d48b88

msmirnov reviewed Nov 9, 2016

View reviewed changes

Comments

Conversation

Pogromistka commented Oct 31, 2016 • edited by honest-hrundel Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

honest-hrundel commented Oct 31, 2016

Uh oh!

honest-hrundel commented Oct 31, 2016

Uh oh!

honest-hrundel commented Oct 31, 2016

Uh oh!

honest-hrundel commented Oct 31, 2016

Uh oh!

honest-hrundel commented Oct 31, 2016

Uh oh!

honest-hrundel commented Nov 1, 2016

Uh oh!

honest-hrundel commented Nov 1, 2016

Uh oh!

honest-hrundel commented Nov 1, 2016

Uh oh!

VasiliiKuznecov commented Nov 2, 2016

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

VasiliiKuznecov commented Nov 2, 2016

Uh oh!

honest-hrundel commented Nov 9, 2016

Uh oh!

honest-hrundel commented Nov 9, 2016

Uh oh!

honest-hrundel commented Nov 9, 2016

Uh oh!

honest-hrundel commented Nov 9, 2016

Uh oh!

honest-hrundel commented Nov 9, 2016

Uh oh!

honest-hrundel commented Nov 9, 2016

Uh oh!

honest-hrundel commented Nov 9, 2016

Uh oh!

honest-hrundel commented Nov 9, 2016

Uh oh!

honest-hrundel commented Nov 9, 2016

Uh oh!

msmirnov commented Nov 9, 2016

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

msmirnov commented Nov 9, 2016

Uh oh!

msmirnov commented Nov 9, 2016

Uh oh!

honest-hrundel commented Nov 9, 2016

Uh oh!

honest-hrundel commented Nov 9, 2016

Uh oh!

msmirnov commented Nov 9, 2016

Uh oh!

msmirnov commented Nov 9, 2016

Pogromistka commented Oct 31, 2016 •

edited by honest-hrundel

Loading