Skip to content

Conversation

@NikGurev
Copy link
Contributor

@NikGurev NikGurev commented Oct 8, 2025

Summary

Set markdown default background as transparent.

@NikGurev NikGurev self-assigned this Oct 8, 2025
@NikGurev NikGurev added the bug Something isn't working label Oct 8, 2025
@github-actions
Copy link

github-actions bot commented Oct 8, 2025

Visit the preview URL for this PR (updated for commit 02649df):

https://koobiq-next--prs-1050-e7d53s1y.web.app

(expires Tue, 14 Oct 2025 09:13:41 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c9e37e518febda70d0317d07e8ceb35ac43c534c

@rmnturov
Copy link
Contributor

rmnturov commented Oct 9, 2025

а зачем вооще белый фон на маркдауне? я правильно понимаю, что у обертки всех маркдаун-элементов есть белый фон? если так, то он не нужен @NikGurev

@NikGurev
Copy link
Contributor Author

NikGurev commented Oct 9, 2025

а зачем вооще белый фон на маркдауне? я правильно понимаю, что у обертки всех маркдаун-элементов есть белый фон? если так, то он не нужен @NikGurev

Тоже задался этим вопросом при выполнении задачи. Цвет background достался по наследству, а сейчас если поменять, то есть вероятность что у кого-то будет отображаться некорректно, поэтому и сделал это поведение настраиваемым

@rmnturov
Copy link
Contributor

rmnturov commented Oct 9, 2025

а зачем вооще белый фон на маркдауне? я правильно понимаю, что у обертки всех маркдаун-элементов есть белый фон? если так, то он не нужен @NikGurev

Тоже задался этим вопросом при выполнении задачи. Цвет background достался по наследству, а сейчас если поменять, то есть вероятность что у кого-то будет отображаться некорректно, поэтому и сделал это поведение настраиваемым

Удали фон, пожулайста. Пусть не будет заливки

@NikGurev NikGurev changed the title fix(markdown): added input for transparent bg (#DS-3909) fix(markdown): сhanged default bg to transparent (#DS-3909) Oct 9, 2025
@NikGurev NikGurev merged commit ed6753a into main Oct 9, 2025
4 checks passed
@NikGurev NikGurev deleted the bugfix/DS-3909 branch October 9, 2025 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants