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

Переносит рендеринг снега в воркер #1318

Closed
wants to merge 1 commit into from

Conversation

Chudesnov
Copy link

@Chudesnov Chudesnov commented Dec 26, 2024

Не решает какую-то конкретную проблему, я просто посмотрел на хорошую поддержку https://developer.mozilla.org/en-US/docs/Web/API/OffscreenCanvas#browser_compatibility в браузерах, и мне показалось хорошей идеей унести рендеринг из главного UI-потока, так как работу с canvas в отличие от предыдущего решения на CSS сам браузер оптимизировать и вынести на отдельное ядро или тем более GPU не способен.

@Chudesnov Chudesnov marked this pull request as draft December 26, 2024 20:49
Copy link

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

@Chudesnov Chudesnov marked this pull request as ready for review December 27, 2024 00:51
@@ -63,7 +63,7 @@ const sw = () => {

const scripts = () => {
return gulp
.src('src/scripts/index.js')
.src(['src/scripts/index.js', 'src/scripts/workers/snow-worker-25.js'])
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Не уверен, что этого достаточно, чтобы файл оказался где надо во всех режимах, система сборки у вас довольно сложная, буду рад советам.

@@ -1,5 +1,5 @@
{
"name": "platform",
"name": "doka-platform",
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"name": "doka-platform",
"name": "platform",

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

случайно закоммитилось

@solarrust
Copy link
Member

Привет!
Спасибо большое за эту интересную оптимизацию.
Снег — временная фишка на пару недель. Мы всей командой проверяли на тормоза на разных устройствах и не нашли проблем. От пользователей тоже не поступало обратной связи по этому поводу.
Кажется, ты предлагаешь улучшение ради улучшения. На текущий функционал это никак не повлияет с точки зрения пользователя. Давай оставим этот маленький милый прикол таким, какой он есть.

Но мне очень нравится твой код и твой подход. Кажется, что из этого получился бы отличный рецепт! Что скажешь? Попробуешь написать?
Или можем сделать это в соавторстве. Я собиралась описать процесс создания снега на канвасе. А ты мог бы дописать свою часть. Как тебе такая идея?
Дай знать, какой из вариантов авторства тебе нравится больше =)
А этот пулреквест закрою.

@solarrust solarrust closed this Dec 27, 2024
@Chudesnov
Copy link
Author

@solarrust да, я тоже подумал в процессе, что улучшение тут вряд ли пригодится, но ПР решил отправить все равно. По поводу рецепта подумаю, спасибо.

@Chudesnov
Copy link
Author

Кстати, role=presentation на канвас я бы таки поставил, а то сейчас он отображается как неподписанное (не декоративное) изображение в дереве доступности.
image

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