Шаламова Лилия #16

LiliyaShalamova · 2016-11-05T18:57:00Z

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

honest-hrundel · 2016-11-05T18:57:35Z

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

honest-hrundel · 2016-11-05T19:01:27Z

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

honest-hrundel · 2016-11-05T19:08:07Z

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

honest-hrundel · 2016-11-05T19:15:38Z

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

honest-hrundel · 2016-11-05T19:19:44Z

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

LiliyaShalamova · 2016-11-09T09:49:03Z

🍏

savichev-igor

Жаль, что не сделала красивую переключалку, а оставила серый input :<

Также неплохо было бы убрать это дёрганье контента при появлении рамки.

Ещё попробуй пересмотреть цвета, а то этот жёлтый почти не видно на белом фоне.

Ну и вообще сделай интереснее... Вдруг я тебе 50 тысяч отдал за эту страницу, а ты мне такое сверстала... Я бы в суд подал)

savichev-igor · 2016-11-10T15:28:04Z

index.css

@@ -0,0 +1,100 @@
+main
+{
+    width: 900;


А пиксели куда пропали =)?

savichev-igor · 2016-11-10T15:28:38Z

index.css

+
+.category
+{
+    color: darkgrey;


Лучше в hexцвета указывать =)

savichev-igor · 2016-11-10T15:29:06Z

index.html

+    <p>Вывод котиков списком</p>
+    <hr>
+    <main>
+        <div class="block">


Опять div'ы, где семантические тэги :< ?

savichev-igor · 2016-11-10T15:29:25Z

index.html

+                <a href="" class="name">Безумная Бусяяяяяяяяяяяяяяяяяяяяяяяяяяяяяяяяяяяяяяяяя</a><br>
+                <a href="" class="category">Безумный</a>
+                <div class="rating">
+                    <span>&#9734;</span><span>&#9734;</span><span>&#9734;</span><span>&#9734;</span>


С переносами бы хоть их сделала)

savichev-igor · 2016-11-10T15:30:12Z

index.html

+                <p class="cost">$100.99 <span class="cost">$150.99</span></p>
+            </div>
+            <div class="description">
+            Мяяяяяяя&shy;яяяяуууу&shy;уууууу&shy;уууу&shy;ууу&shy;уууууууу&shy;уууу&shy;ууууууууу&shy;ууууу


<div> Я милая киса, и я обещаю всегда делать отступы и учитывать вложенность. </div>

savichev-igor · 2016-11-10T15:30:45Z

index.html

+
+        <div class="block">
+            <div class="photo">
+                <img alt="cat1" src="cats/4.jpg" width="270" height="270" title="Это Алиса">


Ширину и высоту лучше задавать через классы, и вообще нужно указывать только одну сторону, иначе сломаешь пропорции

savichev-igor · 2016-11-10T15:33:08Z

🚀

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

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

LiliyaShalamova · 2016-11-12T06:45:22Z

🍏

meded90 · 2016-11-14T09:51:44Z

index.html

 </head>
 <body>
+    <input type="checkbox" name="string" value="string" class="string">
+    <p>Вывод котиков списком</p>


meded90 · 2016-11-14T09:52:16Z

index.html

+    <p>Вывод котиков списком</p>
+    <hr>
+    <main>
+        <article class="block">


обычно называют item

meded90 · 2016-11-14T09:53:57Z

index.html

+            <section class="card">
+                <a href="" class="name">Лорд</a><br>
+                <a href="" class="category">Пугливый</a>
+                <section class="rating">


section не семантично для данного элемента

meded90 · 2016-11-14T09:55:53Z

index.html

+                </section>
+                <p class="cost">&#036;100.99 <span class="cost">&#036;150.99</span></p>
+            </section>
+            <section class="description">


Задаёт раздел документа, может применяться для блока новостей, контактной информации, глав текста, вкладок в диалоговом окне и др. Раздел обычно содержит заголовок. Допускается вкладывать один тег section внутрь другого.

для текста есть тег p

тоже и к фото

meded90 · 2016-11-14T09:58:51Z

meded90 · 2016-11-14T10:00:11Z

index.css

+span.cost
+{
+    color: #000;
+    font: normal;


чет я незнаю такого значения у font

meded90 · 2016-11-14T10:00:53Z

🍅

honest-hrundel · 2016-11-14T13:45:03Z

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

LiliyaShalamova · 2016-11-14T13:47:02Z

Я делала без доп. задания
Звездочки есть, но состояние не сохраняется при нажатии

meded90 · 2016-11-14T14:01:11Z

meded90 · 2016-11-14T14:01:37Z

meded90 · 2016-11-14T14:03:23Z

пустое место

meded90 · 2016-11-14T14:03:31Z

🍅

honest-hrundel · 2016-11-14T14:19:18Z

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

honest-hrundel · 2016-11-14T14:24:20Z

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

LiliyaShalamova · 2016-11-14T14:24:47Z

🍏

honest-hrundel · 2016-11-14T17:33:54Z

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

LiliyaShalamova · 2016-11-14T17:34:05Z

🍏

trixartem · 2016-11-15T09:50:47Z

Замечания по макету:

Далековато от макета, выглядит не очень красиво. Я бы рекомендовал повторить макет, раз не получилось сделать что-то свое
при сжати экрана начинаю всплывать всяки артефакты, попробуй их избежать
Не увидел эффекта при наведении на фото
На больших экранах все расползатся. Я бы ограничил ширину и выравнял по центру

trixartem

🍅

trixartem · 2016-11-15T09:51:42Z

index.css

+.cat-photo
+{
+    width: 100%;
+    height: auto;


Зачем?

trixartem · 2016-11-15T09:52:43Z

index.css

+.cost
+{
+    color: red;
+    font: bold 110% serif;


ставишь абсолютно любой шрифт семейства? Может задать конкретный шрифт, а потом уже указывать семейство?

trixartem · 2016-11-15T09:56:01Z

index.css

+.list:checked ~ main .item
+{
+    display: block;
+    width: 95%;


почему 95?

trixartem · 2016-11-15T09:56:11Z

index.css

+    margin: 5px;
+    vertical-align: middle;
+    width: 30%;
+    height: auto;


Зачем?

trixartem · 2016-11-15T09:56:59Z

index.html

+    <hr>
+    <main>
+        <article class="item">
+            <div class="photo">


Не нашлось семантичного элемента?подсказка: я рассказывал про него

trixartem · 2016-11-15T09:57:26Z

index.html

+            <section class="card">
+                <a href="" class="name">Безумная Бусяяяяяяяяяяяяяяяяяяяяяяяяяяяяяяяяяяяяяяяяя</a><br>
+                <a href="" class="category">Безумный</a>
+                <div class="rating">


Я бы хотел увидеть звездочки по которым можно нажимать

trixartem · 2016-11-15T09:58:20Z

index.html

+                    <span>&#9734;</span>
+                    <span>&#9734;</span>
+                </div>
+                <p class="cost">&#036;100.99 <span class="cost">&#036;150.99</span></p>


Есть семантичные элемента для удаленных текстов.

trixartem · 2016-11-15T10:02:21Z

index.html

+            <div class="photo">
+                <img alt="cat2" src="cats/2.jpg" title="Это Лорд" class="cat-photo">
+            </div>
+            <section class="card">


Мне кажется, что section не очень здесь подходит, попробуй подобрать другой элемент

Первый

658ad5f

Линтинг

65f4b2b

Линтинг

580e88d

Линтинг

5d27c6f

Линтинг

a77a62d

honest-hrundel assigned savichev-igor Nov 5, 2016

savichev-igor reviewed Nov 10, 2016

View reviewed changes

honest-hrundel assigned meded90 and unassigned savichev-igor Nov 10, 2016

Замечания

f6e2f5a

meded90 reviewed Nov 14, 2016

View reviewed changes

index.html

<p>Вывод котиков списком</p>

<hr>

<main>

<article class="block">

Copy link

meded90 Nov 14, 2016

Choose a reason for hiding this comment

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

обычно называют item

meded90 reviewed Nov 14, 2016

View reviewed changes

index.css

span.cost

{

color: #000;

font: normal;

Copy link

meded90 Nov 14, 2016

Choose a reason for hiding this comment

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

чет я незнаю такого значения у font

Замечания

2bb7c59

Замечания

1488b33

Линтинг

9e53c9c

voychitskaya assigned trixartem and unassigned meded90 Nov 14, 2016

Замечания

f796cff

trixartem reviewed Nov 15, 2016

View reviewed changes

Шаламова Лилия #16

Are you sure you want to change the base?

Шаламова Лилия #16

Conversation

LiliyaShalamova commented Nov 5, 2016 • edited by honest-hrundel Loading

honest-hrundel commented Nov 5, 2016

honest-hrundel commented Nov 5, 2016

honest-hrundel commented Nov 5, 2016

honest-hrundel commented Nov 5, 2016

honest-hrundel commented Nov 5, 2016

LiliyaShalamova commented Nov 9, 2016

savichev-igor left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

savichev-igor commented Nov 10, 2016

honest-hrundel commented Nov 12, 2016

LiliyaShalamova commented Nov 12, 2016

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

meded90 Nov 14, 2016 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

meded90 commented Nov 14, 2016

Choose a reason for hiding this comment

meded90 commented Nov 14, 2016

honest-hrundel commented Nov 14, 2016

LiliyaShalamova commented Nov 14, 2016

meded90 commented Nov 14, 2016

meded90 commented Nov 14, 2016

meded90 commented Nov 14, 2016

meded90 commented Nov 14, 2016

honest-hrundel commented Nov 14, 2016

honest-hrundel commented Nov 14, 2016

LiliyaShalamova commented Nov 14, 2016

honest-hrundel commented Nov 14, 2016

LiliyaShalamova commented Nov 14, 2016

trixartem commented Nov 15, 2016

trixartem left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

LiliyaShalamova commented Nov 5, 2016 •

edited by honest-hrundel

Loading

meded90 Nov 14, 2016 •

edited

Loading