Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

doc: Help styleguidist to detect proper name for Tabs page #8203

Merged
merged 1 commit into from
Jan 28, 2025

Conversation

andrey-medvedev-vk
Copy link
Contributor

@andrey-medvedev-vk andrey-medvedev-vk commented Jan 28, 2025

В доке, начиная в версиях 7.1.0 7.1.1 пропала страница Tabs. Точнее она есть, но под другим именем TabsModeContext: https://vkcom.github.io/VKUI/7.1.0/#/TabsModeContext

Styleguide перестал адекватно понимать где в файле Tabs именно компонент Tabs.
Можно были либо перенести TabsModeContext из файла Tabs в отдельный, либо добавить displayName в соответсвии с документацией styleguide

Проверил по коду, с остальными компонентами такой проблемы нету.


Явных причин по коду в изменениях между 7.0.1 и 7.1.0 для такого эффекта не заметил, возможно, что что-то в типах поменялось.

Copy link

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.

Copy link
Contributor

github-actions bot commented Jan 28, 2025

size-limit report 📦

Path Size
JS 393.71 KB (+0.01% 🔺)
JS (gzip) 119.4 KB (+0.01% 🔺)
JS (brotli) 98.29 KB (-0.07% 🔽)
JS import Div (tree shaking) 1.56 KB (0%)
CSS 346.25 KB (0%)
CSS (gzip) 42.89 KB (0%)
CSS (brotli) 34.26 KB (0%)

@andrey-medvedev-vk andrey-medvedev-vk added this to the v7.1.2 milestone Jan 28, 2025
@andrey-medvedev-vk andrey-medvedev-vk marked this pull request as ready for review January 28, 2025 11:27
@andrey-medvedev-vk andrey-medvedev-vk requested a review from a team as a code owner January 28, 2025 11:27
Copy link
Contributor

e2e tests

Playwright Report

Copy link
Contributor

👀 Docs deployed

Commit 2b93483

Copy link

codecov bot commented Jan 28, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.51%. Comparing base (f2f7055) to head (2b93483).
Report is 8 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #8203   +/-   ##
=======================================
  Coverage   95.50%   95.51%           
=======================================
  Files         403      403           
  Lines       11495    11515   +20     
  Branches     3807     3819   +12     
=======================================
+ Hits        10978    10998   +20     
  Misses        517      517           
Flag Coverage Δ
unittests 95.51% <100.00%> (+<0.01%) ⬆️

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 andrey-medvedev-vk merged commit 9b0cfa2 into master Jan 28, 2025
52 of 54 checks passed
@andrey-medvedev-vk andrey-medvedev-vk deleted the mendrew/doc/fix-tabs-page-name branch January 28, 2025 13:58
andrey-medvedev-vk added a commit that referenced this pull request Jan 28, 2025
Исправляем отображение `Tabs` страницы в версиях доки v7.1.0 и v7.1.1, так как сейчас в этих версиях страница отображается как https://vkcom.github.io/VKUI/7.1.0/#/TabsModeContext и пропсы соответствуют `TabsModeContext`.

Применил патч из #8203 конкретно к коду v7.1.0 и v7.1.1, для каждой версии пересобрал `styleguide` и результат добавил в ветку `gh-pages`.
@vkcom-publisher
Copy link
Contributor

v7.1.2 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

4 participants