fix(Group): refactor styles for fix adaptivity

[inomdzhon]

• close [Bug]: AdaptivityProvider по-умолчанию на vkcom имеет особую отрисовку Group в ModalPage #4656

---

• Дизайн-ревью

Описание

• реструктурировал CSS файл, чтобы было понятней что к чему;
• размеры заменил на токены из @vkonktate/vkui-tokens;
• отказался от использования <Separator /> из-за того, что его задевает vkuiInternalModalPage, вместо этого объединил Group__separator--spacing и Group__separator--separator в один элемент Group__separator;
• добавил модификатор Group--sizeX--regular, т.к. он определяет mode как "card" при sizeX="regular", если mode === "none";
• вынес <Separator /> из <div /> в ModalPageHeader.tsx, чтобы боковые отступы не влияли на размеры сепаратора.

Дизайн

Внутри ModalPage

При platform="android"/platform="ios" и sizeX="regular" в ModalPageHeader удаляем отступы у Separator.

до и после

---

• related to fix(Group): adaptivity #5344, fix(Group): spacing adaptivity #5364, Fix Group padding for modal #5371, feat(AppRoot): add layout #5642

[github-actions]

size-limit report 📦

Path	Size
JS	380.44 KB (+0.02% 🔺)
JS (gzip)	116.03 KB (+0.03% 🔺)
JS (brotli)	95.44 KB (-0.02% 🔽)
JS import Div (tree shaking)	1.39 KB (0%)
CSS	307.19 KB (+0.41% 🔺)
CSS (gzip)	39.37 KB (+0.28% 🔺)
CSS (brotli)	31.63 KB (+0.36% 🔺)

[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 78c59e5

[codecov]

Codecov Report

Attention: Patch coverage is 53.84615% with 6 lines in your changes missing coverage. Please review.

Project coverage is 83.54%. Comparing base (a5ffcd8) to head (78c59e5).
Report is 2 commits behind head on master.

Files	Patch %	Lines
packages/vkui/src/components/Group/Group.tsx	45.45%	6 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #7284      +/-   ##
==========================================
+ Coverage   83.45%   83.54%   +0.09%     
==========================================
  Files         369      369              
  Lines       11175    11152      -23     
  Branches     3656     3643      -13     
==========================================
- Hits         9326     9317       -9     
+ Misses       1849     1835      -14     

Flag	Coverage Δ
unittests	83.54% <53.84%> (+0.09%)	⬆️

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.

[Zaycevq]

👍🏽

[mendrew]

Класс 👍

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