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

[Bug][Gallery]: Фатальная ошибка если галерея не имеет потомков при наличии флага looped #7672

Closed
Kritolan opened this issue Sep 28, 2024 · 7 comments · Fixed by #7686

Comments

@Kritolan
Copy link

Описание

Если в галерея в зацикленном режиме - то в случае, когда в ней отсутствуют потомки т.е. показывать нечего - вылетает ошибка ломающая всю страницу. Сама ошибка возникает в дочернем компоненте CarouselBase, который применяется для зацикленного режима.

Версия

6.7.0

В каких браузерах воспроизводится проблема?

No response

Шаги воспроизведения

Добавить в любое место страницы <Gallery looped />

Ожидаемое поведение

Компонент просто не будет отображаться.

Скриншоты

изображение

Пример с воспроизведением

No response

@BlackySoul
Copy link
Contributor

Привет! Спасибо за issue, ошибку исправим и, возможно, в мажорной версии сделаем проп children обязательным. Какое практическое применение у Gallery без слайдов?

@BlackySoul BlackySoul self-assigned this Sep 30, 2024
@Kritolan
Copy link
Author

Ну например в случае, когда содержимое загружается динамически и в итоге может отсутствовать. Без флага looped - компонент просто не рендерится и в целом все выглядит нормально.

@BlackySoul
Copy link
Contributor

Ну например в случае, когда содержимое загружается динамически и в итоге может отсутствовать. Без флага looped - компонент просто не рендерится и в целом все выглядит нормально.

Ага, в обычном режиме сейчас вместо галереи ничего не отрисовывается, но, кажется, что это плохо само по себе - layout shift может быть существенный, когда слайды подгрузятся. Лучше будет отрисовывать какой-то слайд-заглушку, чтобы этот момент минимизировать + дать пользователю знать о наличии контента.

@inomdzhon
Copy link
Contributor

Лучше будет отрисовывать какой-то слайд-заглушку, чтобы этот момент минимизировать + дать пользователю знать о наличии контента.

Дополню, что есть компонент Skeleton

@Kritolan
Copy link
Author

Kritolan commented Sep 30, 2024

Конкретно этот момент мне кажется лучше оставить на совести автора, как и с любыми другими обертками. Т.е. да, подобное поведение плохо - но это уже не отвественность библиотеки.

@BlackySoul
Copy link
Contributor

Конкретно этот момент мне кажется лучше оставить на совести автора, как и с любыми другими обертками.

Ага, ошибку в любом случае поправим х)

@vkcom-publisher
Copy link
Contributor

v7.0.0-beta.0 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

4 participants