fix(Snackbar/AppRootPortal): fix default shouldDisablePortalLogic in AppRootPortal

[mendrew]

• close [Bug]: Snackbar перекрывает Tabbar при появлении #6380

---

Описание

В рефакторинге AppRootPortal в c7e4c41 мы инверсировали логику использования портала и там закралась ошибка в значении по умолчанию. До рефакторинга мы использовали портал если mode !== 'full',

VKUI/packages/vkui/src/components/AppRoot/AppRootPortal.tsx

Lines 35 to 42 in 7f1888d

	const forcePortal = forcePortalProp ?? mode !== 'full';
	const portalContainer = getPortalContainer(portalRootProp, portalRoot);
	const ignoreDisablePortalFlagFromContext = portalRootProp && forcePortal;
	const shouldUsePortal = ignoreDisablePortalFlagFromContext
	? true
	: !disablePortal && portalContainer && forcePortal;

а теперь выключаем портал если mode !== 'full'.

VKUI/packages/vkui/src/components/AppRoot/AppRootPortal.tsx

Lines 38 to 51 in 959611e

	function shouldDisablePortal(
	usePortal: AppRootPortalProps['usePortal'],
	mode: AppRootContextInterface['mode'],
	disablePortal: boolean,
	) {
	if (usePortal !== undefined) {
	if (typeof usePortal !== 'boolean') {
	return false;
	}
	return disablePortal || usePortal !== true;
	}
	// fallback
	return disablePortal || mode === 'full';
	}

Эта ошибка повлекла за собой по крайней мере неверное позиционирование Snackbar, когда речь идёт о Snackbar + Tabbar у Epic в режиме миниапов (mode="full"). Snackbar рендерился используя портал, тем самым выпадая из селектора, задающего отступ.

Возможно, что нам тут даже не стоит полагаться на селектор вовсе и читать информацию о наличии tabbar из контекста, определяя размер экрана в котором tabbar рендерится внизу. В то же время Epic это больше о мобильных экранах.
Тем не менее ошибка в AppRootPortal есть.

Изменения

• Поправил условие для режима full по умолчанию. Покрыл тестами.
• обернул children в React.Fragment чтобы ts не ругался на типы AppRootPortal.

'AppRootPortal' cannot be used as a JSX component.
Its return type 'ReactNode' is not a valid JSX element. (tsserver 2786)

[github-actions]

size-limit report 📦

Path	Size
JS	347.98 KB (+0.01% 🔺)
JS (gzip)	106.44 KB (+0.01% 🔺)
JS (brotli)	87.94 KB (-0.08% 🔽)
JS import Div (tree shaking)	1.43 KB (0%)
CSS	256.43 KB (0%)
CSS (gzip)	33.6 KB (0%)
CSS (brotli)	27.29 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.

Latest deployment of this branch, based on commit f5513f6:

Sandbox	Source
VKUI TypeScript	Configuration

[github-actions]

e2e tests

Playwright Report

[github-actions]

👀 Docs deployed

• ✅ Styleguide
• ✅ Storybook

Commit f5513f6

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (c2f2079) 81.29% compared to head (f5513f6) 81.42%.
Report is 6 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #6389      +/-   ##
==========================================
+ Coverage   81.29%   81.42%   +0.13%     
==========================================
  Files         326      326              
  Lines       10127    10113      -14     
  Branches     3406     3401       -5     
==========================================
+ Hits         8233     8235       +2     
+ Misses       1894     1878      -16     

Flag	Coverage Δ
unittests	81.42% <100.00%> (+0.13%)	⬆️

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.

[scffs]

Отдельное спасибо за подробное описание проблемы

[inomdzhon]

🚀

[vkcom-publisher]

✅ v6.0.1 🎉