fix(Link): Remove ripple part of Link by disabling Tappable hoverMode

[mendrew]

• close [Bug]: Link в Safari внутри текста с переполнением выглядывает наружу, хотя должен исчезать вместе с текстом #7086

---

Описание

Из-за того, что у компонентов Link всегда рендерится часть Tappable для ripple-эффекта, спозиционированная абсолютно, Link не прячется в Safari вместе с контентом если контент переполнил родительский элемент.

Это, конечно, странное поведение Safari, но и у нас довольно странно в DOM вставлять Ripple компонент, который для Link совсем не нужен.

Изменения

Руководствуясь условием в Tappable, по которому мы рендерим Ripple компонент

VKUI/packages/vkui/src/components/Tappable/Tappable.tsx

Lines 74 to 76 in 92d8c08

	{isClickable && (hoverMode === 'background' || activeMode === 'background') && (
	<Ripple needRipple={needRipple} clicks={clicks} />
	)}

мы убираем hoverMode="none" у Link.

Это безопасно, потому что hoverMode="background" у Tappable есть по умолчанию и управляет он фоном, которого у Link нету. Так что на Link это никак не повлияет.

VKUI/packages/vkui/src/components/Tappable/Tappable.module.css

Lines 64 to 67 in 92d8c08

	.Tappable--hovered-background > .Tappable__stateLayer {
	background-color: var(--vkui--color_transparent--hover);
	}

VKUI/packages/vkui/src/components/Link/Link.module.css

Line 4 in 92d8c08

background: none;

Изображения

[github-actions]

size-limit report 📦

Path	Size
JS	365.03 KB (+0.01% 🔺)
JS (gzip)	111.77 KB (+0.01% 🔺)
JS (brotli)	91.95 KB (-0.19% 🔽)
JS import Div (tree shaking)	1.42 KB (0%)
CSS	286.55 KB (0%)
CSS (gzip)	37.18 KB (0%)
CSS (brotli)	30.05 KB (0%)

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

[github-actions]

e2e tests

Playwright Report

[github-actions]

👀 Docs deployed

• ✅ Styleguide
• ✅ Storybook

Commit 92d8c08

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.67%. Comparing base (606ec64) to head (92d8c08).

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #7088   +/-   ##
=======================================
  Coverage   83.67%   83.67%           
=======================================
  Files         353      353           
  Lines       10565    10565           
  Branches     3499     3499           
=======================================
  Hits         8840     8840           
  Misses       1725     1725           

Flag	Coverage Δ
unittests	83.67% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[inomdzhon]

👏