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

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

Open
EgorPopovPP opened this issue Dec 18, 2019 · 7 comments
Open

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

EgorPopovPP opened this issue Dec 18, 2019 · 7 comments

Comments

@EgorPopovPP
Copy link

Читаю описание блока 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
Copy link
Owner

yndx-shri-reviewer commented Dec 19, 2019

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

@rakov-di
Copy link

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

@yndx-shri-reviewer
Copy link
Owner

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

@rakov-di
Copy link

rakov-di commented Jan 8, 2020

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

@Stepan47
Copy link

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

@juliaovod
Copy link

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

@Stepan47
Copy link

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

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

No branches or pull requests

5 participants