Skip to content

feat(gallery): добавлена возможность скрывать кнопку поделиться#1956

Open
DavidGuryanov wants to merge 6 commits intocore-ds:masterfrom
DavidGuryanov:DS-14162
Open

feat(gallery): добавлена возможность скрывать кнопку поделиться#1956
DavidGuryanov wants to merge 6 commits intocore-ds:masterfrom
DavidGuryanov:DS-14162

Conversation

@DavidGuryanov
Copy link
Contributor

Раньше можно было скрывать только кнопку "скачать". Теперь, после моей инновационной доработки, можно скрывать ещё и кнопку "поделиться".
Во-первых, это добавляет гибкости компоненту.
Во-вторых, в некоторых вебвью не работает navigator.share, так что эта кнопка не будет работать, соответсвенно её надо скрывать от пользователя.

Чек лист

  • Задача сформулирована и описана в JIRA
  • В названии ветки есть айдишник задачи в JIRA (fix/DS-1234), ссылку прикреплять не надо
  • У реквеста осмысленное название feat(...) или fix(...) по conventional commits (https://www.conventionalcommits.org)
  • Код покрыт тестами и протестирован в различных браузерах
  • Добавленные пропсы добавлены в демки и описаны в документации
  • К реквесту добавлен changeset

Если есть визуальные изменения

  • Прикреплено изображение было/стало

@changeset-bot
Copy link

changeset-bot bot commented Nov 21, 2025

🦋 Changeset detected

Latest commit: 8f51d1d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@alfalab/core-components-gallery Minor
@alfalab/core-components Minor

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

@coveralls
Copy link

coveralls commented Nov 21, 2025

Pull Request Test Coverage Report for Build 20958106631

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 69 of 73 (94.52%) changed or added relevant lines in 12 files are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage increased (+0.1%) to 79.829%

Changes Missing Coverage Covered Lines Changed/Added Lines %
packages/select/src/components/search/Component.tsx 9 10 90.0%
packages/select/src/hooks/use-native-scrollbar.ts 13 14 92.86%
packages/gallery/src/components/header-mobile/Component.tsx 0 2 0.0%
Files with Coverage Reduction New Missed Lines %
packages/select/src/components/virtual-options-list/Component.tsx 1 69.17%
Totals Coverage Status
Change from base Build 19334456547: 0.1%
Covered Lines: 9722
Relevant Lines: 11411

💛 - Coveralls

@fulcanellee
Copy link
Contributor

@Oladii @temss-front посмотрите пожалуйста

@fulcanellee
Copy link
Contributor

@Oladii @temss-front

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants