search

yndx-shri-reviewer / shri-2020-task-1

Задание 1. Реализуйте дизайн-систему — ШРИ, Москва, 2020
https://yandex.ru/promo/academy/shri
44 stars 103 forks source link

Border в блоке card > image #32

Open EgorPopovPP opened 4 years ago

EgorPopovPP commented 4 years ago

Читаю описание блока image. [https://github.com/uporot1k/shri-2020-task-1/blob/master/BASE.md#%D0%B1%D0%BB%D0%BE%D0%BA-image]()

Блок служит для отображения изображений. Растягивается на всю ширину, имеет сплошную границу шириной 3px и содержит заглушку. Заглушка, цвета фона и границы задаются темой или стилями контентного блока.

По дефолту, цвет border должен браться из соответствующей переменной дефолтной темы. В случаях, когда какому либо из родительских блоков присваивается одна из тем, цвет границы должен браться уже из нее.

Прилигаю скриншот для пояснения (http://prntscr.com/qcgo5y)

Белым цветом выделены несоответствия картинки и исходной верстки. На данном скриншоте четко видно что цвет border у блока image не соответствует макету.

У темы .theme_color_project-default цвет border = #000000 , а у темы .theme_color_project-inverse - #ffffff.

В этом кейсе идет вот такая верстка http://prntscr.com/qcgre3 На скришоте видно, что родительскому блоку image присваивается тема .theme_color_project-inverse

Я не понимаю как цвет border у блока image может быть черного цвета, если у родителя задана инверсная тема, цвет border у которой белый

yndx-shri-reviewer commented 4 years ago

К сожалению, ответ на этот вопрос будет явной подсказкой по реализации, поэтому мы не можем дать на него ответ. CSS позволяет достаточно просто добиться нужного результата.

rakov-di commented 4 years ago

@yndx-shri-reviewer А можете ли дать ответ на вопрос, действительно ли тут должна быть рамка и действительно ли она должна быть черного цвета?

yndx-shri-reviewer commented 4 years ago

Прошу прощения за долгий ответ. А каким образом наличие или отсутствие рамки мешает реализации контентного блока?

rakov-di commented 4 years ago

Не мешает. Понятно, что рамку можно убирать или перекрашивать и переопределять как душе угодно. Просто любое поведение не описанное четко в ТЗ, или любое расхождение в верстке в несколько пикселей по результатам автотестов (как в вопросе @uporot1k) вызывает сомнения - сделано ли это специально, чтобы посмотреть, как человек справится с задачей, или это опечатка, случайная ошибка в задании/скриншоте и т.д. Имею ввиду, что некоторое кол-во таких "расхождений" (в первом задании) проще править изменением строчки в ТЗ, чем в коде)) Поэтому на всякий случай мы и уточняем. Конкретно для меня этот вопрос не актуален, т.к. задание все равно уже сдал на проверку.

Stepan47 commented 4 years ago

Можно вопрос? Может это покажется и глупым, но вам отправлять мою страницу надо, или нет? как-то вы ее будете проверять? если да, то как отправить? Извините, я просто в Github недавно. Я сделал пока только первое задание,

juliaovod commented 4 years ago

@Stepan47 На сайте https://account.shri.yandex/ ты должен был заполнить анкету и после этого появляется возможность загрузить свой репозиторий на проверку автотестам. И после каждой проверки (автотестирования) у тебя в репозитории проекта создается ветка с результатами, там в папке screenshots будет сравнение макетов твоего результата и того, который должен быть.

Stepan47 commented 4 years ago

@juliaovod, спасибо, помогли.