Skip to content

Comments

Автоматическое включение и выключение снега#1322

Closed
besdar wants to merge 2 commits intodoka-guide:mainfrom
besdar:automatic-snow
Closed

Автоматическое включение и выключение снега#1322
besdar wants to merge 2 commits intodoka-guide:mainfrom
besdar:automatic-snow

Conversation

@besdar
Copy link

@besdar besdar commented Jan 23, 2025

Привет @solarrust!

Я заметил, что вы сейчас на сайте Доки вручную включаете и выключаете анимацию снега на сайте в новогодние праздники. Как я понял из истории коммитов, включать надо 25 декабря и выключать 15 января. Предлагаю автоматизировать этот функционал и более не вспоминать об этом :)

P.S. Я вижу в DOM добавляемый элемент canvas (#snowCanvas) для снега, но он и так скрытый и я не вижу, чтобы он как-то нарушал UX, так что я его не трогал, но возможно с ним нужно дополнительно что-то сделать.

@github-actions
Copy link

Превью контента из 5ab7524 опубликовано.

@igsekor
Copy link
Member

igsekor commented Jan 28, 2025

Привет @besdar !

Прошу прощения за то, что врываюсь....

Спасибо большое за PR ❤️

Мы не каждый год делаем снег на сайте. Первый раз мы сделали через год после того, как вышли в прод. Это было интересное и олдскульное решение, навеянное ностальгией. Кроме того, это было интересное решение прямо у нас на сайте в поддержку первого рецепта в новом тогда разделе «Рецепты». В этот раз мы включили снег в новом исполнении с точки зрения перформанса. Это более современное решение, которое тоже внесло определённую ясность в то, что одну и ту же фишку можно реализовать разными способами, с разной степенью реализма и разной производительностью. Снег в этом году был совсем иным по качеству, а стало быть, мог помочь кому-то, кто задумывался о реализации чего-то подобного.

Я закрою этот PR, поскольку подобной автоматизации нам не нужно. Мы делаем каждый раз всё вручную. Но всё равно спасибо! Мы очень ценим вклад каждого нашего участника!

@igsekor igsekor closed this Jan 28, 2025
@besdar besdar deleted the automatic-snow branch January 29, 2025 07:58
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

Successfully merging this pull request may close these issues.

2 participants