a11y(Card): change default tags to components Card, CardGrid, CardScroll, ContentCard

[EldarMuhamethanov]

• close [BREAKING CHANGE][Feature]: Заменить теги по умолчанию в карточках #7336

---

• e2e-тесты
• Release notes

Описание

Нужно заменить теги по умолчанию для рендеринга компонентов Card, CardGrid, CardScroll, ContentCard на семантические

Изменения

Заменил теги по умолчанию у компонентов Card, CardGrid, CardScroll, ContentCard на ul или li

Release notes

BREAKING CHANGE

• Card: изменен тег используемый по умолчанию для ренедеринга компонента с div на li
• ContentCard: изменен тег используемый по умолчанию для ренедеринга компонента с div на li
• CardGrid: изменен тег используемый по умолчанию для ренедеринга компонента с div на ul
• CardScroll: изменен тег используемый по умолчанию для ренедеринга компонента с div на ul

[github-actions]

size-limit report 📦

Path	Size
JS	385.3 KB (-0.01% 🔽)
JS (gzip)	116.5 KB (0%)
JS (brotli)	95.67 KB (-0.03% 🔽)
JS import Div (tree shaking)	1.45 KB (0%)
CSS	330.78 KB (0%)
CSS (gzip)	41.39 KB (0%)
CSS (brotli)	32.85 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 5e6e72b

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.21%. Comparing base (cc8faef) to head (5e6e72b).
Report is 16 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #7520   +/-   ##
=======================================
  Coverage   95.21%   95.21%           
=======================================
  Files         384      384           
  Lines       11345    11345           
  Branches     3720     3720           
=======================================
  Hits        10802    10802           
  Misses        543      543           

Flag	Coverage Δ
unittests	95.21% <100.00%> (ø)

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.

[andrey-medvedev-vk]

Классно!

Я бы ещё предложил пройтись по страницам компонентов в доке и чуть текст поменять, явно говоря в секции по a11y, что у нас по умолчанию уже такие теги используются.

Чтобы пользвателям не надо было лезть в пропсы и искать значение по умолчанию там, после того, как валидатор html выдаст ошибку, что внезапно где-то есть тег li без ul.

[EldarMuhamethanov]

Классно!

Я бы ещё предложил пройтись по страницам компонентов в доке и чуть текст поменять, явно говоря в секции по a11y, что у нас по умолчанию уже такие теги используются.

Чтобы пользвателям не надо было лезть в пропсы и искать значение по умолчанию там, после того, как валидатор html выдаст ошибку, что внезапно где-то есть тег li без ul.

Точно, совсем забыл это поменять. Теперь поправил документацию

[andrey-medvedev-vk]

💅