Skip to content

fix(base-modal): content height in Safari 26.1#2162

Open
mitrofanzxc wants to merge 1 commit intomasterfrom
feature/PORTALAPI-1029
Open

fix(base-modal): content height in Safari 26.1#2162
mitrofanzxc wants to merge 1 commit intomasterfrom
feature/PORTALAPI-1029

Conversation

@mitrofanzxc
Copy link
Copy Markdown

@mitrofanzxc mitrofanzxc commented Apr 7, 2026

fix: Исправление растягивания контента модального окна в Safari

В Safari 26.1 наблюдается баг рендеринга: при использовании height: 100% в классе .content компонента BaseModal, содержимое модального окна некорректно растягивалось на всю высоту экрана, если общая высота контента была значительной. В Google Chrome поведение оставалось корректным.

Что сделано:

  • Внедрен CSS-хак, специфичный для движка WebKit (Safari), который переопределяет height: 100% на height: auto только для этого браузера.
  • Это позволяет сохранить корректное поведение и обратную совместимость во всех остальных браузерах (Chrome, Firefox, Edge), не нарушая сложную верстку зависимых компонентов, таких как UniversalModal и SidePanel.
  • Решение протестировано: визуальные регрессии в Chrome отсутствуют (скриншотные тесты проходят), баг в Safari исправлен.

Чек лист

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

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

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

Было:
image

Стало:
image

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 7, 2026

🦋 Changeset detected

Latest commit: 08e35a2

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

This PR includes changesets to release 26 packages
Name Type
@alfalab/core-components-base-modal Patch
@alfalab/core-components-bottom-sheet Patch
@alfalab/core-components-drawer Patch
@alfalab/core-components-gallery Patch
@alfalab/core-components-modal Patch
@alfalab/core-components-popup-sheet Patch
@alfalab/core-components Patch
@alfalab/core-components-select Patch
@alfalab/core-components-side-panel Patch
@alfalab/core-components-universal-modal Patch
@alfalab/core-components-select-with-tags Patch
@alfalab/core-components-tooltip Patch
@alfalab/core-components-calendar Patch
@alfalab/core-components-custom-picker-button Patch
@alfalab/core-components-input-autocomplete Patch
@alfalab/core-components-international-phone-input Patch
@alfalab/core-components-intl-phone-input Patch
@alfalab/core-components-picker-button Patch
@alfalab/core-components-table Patch
@alfalab/core-components-calendar-input Patch
@alfalab/core-components-calendar-range Patch
@alfalab/core-components-calendar-with-skeleton Patch
@alfalab/core-components-date-range-input Patch
@alfalab/core-components-date-time-input Patch
@alfalab/core-components-universal-date-input Patch
@alfalab/core-components-tabs Patch

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
Copy Markdown

coveralls commented Apr 7, 2026

Coverage Report for CI Build 24082926206

Coverage remained the same at 81.916%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 11605
Covered Lines: 9584
Line Coverage: 82.59%
Relevant Branches: 2175
Covered Branches: 1704
Branch Coverage: 78.34%
Branches in Coverage %: Yes
Coverage Strength: 230.15 hits per line

💛 - Coveralls

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2026

Demo build (default)

https://core-ds.github.io/core-components/2162

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2026

Demo build (alfasans)

https://core-ds.github.io/core-components/2162-alfasans

@mitrofanzxc mitrofanzxc force-pushed the feature/PORTALAPI-1029 branch from f881bcb to 08e35a2 Compare April 7, 2026 13:06
Фикс для Safari 26.1:
Заменяем height: 100% на auto, чтобы контент не растягивался на весь экран.
*/
@supports (background: -webkit-named-image(apple-pay-logo-black)) {
Copy link
Copy Markdown
Collaborator

@hextion hextion Apr 9, 2026

Choose a reason for hiding this comment

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

@mitrofanzxc подскажи пож-та почему такое условие и какую проблему решаешь?

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.

4 participants