Conversation
AlekseyLexey
requested review from
SiebenSieben,
dHIM24,
fulcanellee and
hextion
as code owners
🦋 Changeset detectedLatest commit: 18e3dc2 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Bundle size report
|
Pull Request Test Coverage Report for Build 21629301803Details
💛 - Coveralls |
|
@temss-front проверь пожалуйста |
|
@fulcanellee Да, они пересекаются (одна и та же таска), но облегченная версия библиотеки не весь функционал имеет (текст на скрине). На демо пропал |
| async (attempt = 1, maxAttempts = 3): Promise<typeof Hls | null> => { | ||
| try { | ||
| const { default: HlsLib } = await import( | ||
| /* webpackChunkName: "hls-js-gallery" */ 'hls.js' |
There was a problem hiding this comment.
может лучше взять hls.js/dist/hls.min.js? он вроде поменьше размером. или добавить флажек, какой грузить...
There was a problem hiding this comment.
-
По поводу hls.js/dist/hls.min.js — переход на использование только этой версии (лёгкой) нам не подходит: пропадают субтитры (скрин выше, можно собрать демо и посмотреть).
-
По поводу флага — такой подход тоже рассматривал, но у него есть минусы:
-
Пользователь галереи не владеет контентом: сейчас субтитров может не быть и стоит флаг на лёгкую версию, а если их потом добавят в поток — субтитры не появятся, пока флаг не сменим.
-
Если на странице есть и «полная», и «лёгкая» галереи, при открытии видео в обеих подгрузятся оба чанка.
There was a problem hiding this comment.
2 галереи ето конечно плохо да.
но цена субтитров конечно запредельная :(
fulcanellee
left a comment
fulcanellee
left a comment
There was a problem hiding this comment.
Также нужен апрув от @temss-front
Co-authored-by: fulcanellee <45999900+fulcanellee@users.noreply.github.com>
Co-authored-by: fulcanellee <45999900+fulcanellee@users.noreply.github.com>
|
@temss-front нужно твою ревью |
5 participants
Заменен статический импорт hls.js на динамический.
- hls.js вынесен в отдельный чанк (hls-js-gallery)
- Также добавлена логика ретрая.
До:
После:
