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

add color background to language images #348

Merged
merged 2 commits into from Jan 11, 2023

Conversation

alexkwyk
Copy link
Contributor

@alexkwyk alexkwyk commented Dec 23, 2022

closes #329

@fey
Copy link
Contributor

fey commented Dec 27, 2022

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

@alexkwyk alexkwyk marked this pull request as draft December 27, 2022 21:30
@alexkwyk alexkwyk marked this pull request as ready for review December 28, 2022 19:42
@alexkwyk
Copy link
Contributor Author

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

@fey
Copy link
Contributor

fey commented Dec 30, 2022

@alexkwyk вы можете сделать ребейз своей ветки, убрав коммит и сделав git pull --rebase upstream main, чтобы стянуть изменения из репозитория Хекслета.

@alexkwyk alexkwyk force-pushed the fix-sharing-preview branch 3 times, most recently from de49762 to ce88bd2 Compare December 30, 2022 16:30
@alexkwyk alexkwyk closed this Dec 30, 2022
@alexkwyk alexkwyk reopened this Dec 30, 2022
@alexkwyk
Copy link
Contributor Author

извиняюсь, что-то я тут намудрил, теперь вроде бы все хорошо

@alexkwyk alexkwyk changed the title Change image source for social media sharing add color background to language images Dec 30, 2022
@amshkv
Copy link
Contributor

amshkv commented Dec 30, 2022

если картинки не прозрачные, то зачем png, если есть jpg, картинка наверное будет меньше в размере
и Коля, может нам картинки в языке хранить, чтобы ничего не падало?

@fey
Copy link
Contributor

fey commented Dec 30, 2022

Чтоб картинки хранить в языках, это надо код писать!
А непрозрачные картинки не заставляют писать код, плюс можно перейти на картинки в языках.

@alexkwyk
Copy link
Contributor Author

про форматы и не подумал, переделать на jpg?

@amshkv
Copy link
Contributor

amshkv commented Dec 31, 2022

если при этом размер файла уменьшиться, то да, переделать

@alexkwyk
Copy link
Contributor Author

alexkwyk commented Dec 31, 2022

в jpg почему-то размер выше раза в 3-4 даже при очень сильном сжатии, в png получилось пересохранить на размер меньше, делал в фотошопе

@acidmaksim
Copy link

@amshkv посмотри, пожалуйста

@amshkv amshkv merged commit 6bc2e81 into hexlet-basics:main Jan 11, 2023
@amshkv
Copy link
Contributor

amshkv commented Jan 11, 2023

я задеплоил, но попрошу пару моментов поправить:
1 - на сайте всё еще используются какие-то цвета, а они уже не нужны и их можно выпилить - https://github.com/hexlet-basics/hexlet-basics/blob/main/app/views/web/shared/_languages.html.slim#L5
2 - там у карточки стоят паддинги, которые можно убрать, а вот в саму картинку как будто эти отступы можно добавить (чтобы подальше от краев было)

пысы: жаву как-будто сильно порезало

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Исправить отображение картинок в превью
4 participants