refactor(AppRoot): fix appearance cleanup for mode="embedded" #6263
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
h2. Описание Выполнил TODO: нужно было перенести установку `appearance` из `ConfigProvider` в `AppRoot`. h2. Изменения - `AppRoot` - Перенёс все проверки и модификации контейнеров в один эффект. - В файл `helpers.ts` перенёс функции (вместе с их константами), а также вынес некоторую логику в функции. - В `types.ts` хранятся типы, которые шарятся между `helpers.ts` и `AppRoot.tsx`. - Удалил установку `color-scheme` на `documentElement`, т.к. она уже есть в `styles/common.css`. - Переделал `portalRoot` с `useState()` на `useRef()`, тем самым не будет двойного рендера. - `AppRootContext` – сделал всё, кроме `layout`, обязательными параметрами, чтобы не приходилост проверять на `undefined`. Завёл константу `DEFAULT_APP_ROOT_CONTEXT_VALUE`, чтобы в тестах можно было делать `Object.assign`, если хочется обернуть в свой контекст. - Дописал документацию для свойств. Для свойства `scroll`, перенёс из `Readme.md` часть про режим скролла. - `AppRoot.test.tsx` – добавил больше тестов, по ходу дела отрефакторил весь файл. - `packages/playwright/vkui/index.css` – заменил CSS селекторе "регулярку" с `$` на `*`, т.к. `$` предполагал, что класс с токеном всегда будет в конце. - `hooks/useObjectMemo.ts` – заменил кастомное сравнение объектов на `isEqual()` из `@vkontakte/vkjs`.
size-limit report 📦
|
|
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 ccedaa3:
|
e2e tests |
👀 Docs deployed
Commit ccedaa3 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6263 +/- ##
==========================================
+ Coverage 81.14% 81.56% +0.41%
==========================================
Files 324 325 +1
Lines 10090 10088 -2
Branches 3386 3378 -8
==========================================
+ Hits 8188 8228 +40
+ Misses 1902 1860 -42
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
mendrew
previously approved these changes
Dec 12, 2023
Co-authored-by: Andrey Medvedev <[email protected]>
Co-authored-by: Andrey Medvedev <[email protected]>
mendrew
approved these changes
Dec 12, 2023
SevereCloud
approved these changes
Dec 13, 2023
inomdzhon
deleted the
imirdzhamolov/issue4810/fix/ConfigProvider-embeded-node
branch
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
embeddedкласс с названием платформы (темы) добавляется наbody#4810Описание
Выполнил TODO: нужно было перенести установку
appearanceизConfigProviderвAppRoot.Изменения
AppRootwindow.helpers.tsперенёс функции (вместе с их константами), а также вынес некоторую логику в функции.types.tsхранятся типы, которые шарятся междуhelpers.tsиAppRoot.tsx.color-schemeнаdocumentElement, т.к. она уже есть вstyles/common.css.portalRootсuseState()наuseRef(), тем самым не будет двойного рендера.AppRootContext– сделал всё, кромеlayout, обязательными параметрами, чтобы не приходилост проверять наundefined. Завёл константуDEFAULT_APP_ROOT_CONTEXT_VALUE, чтобы в тестах можно было делатьObject.assign, если хочется обернуть в свой контекст.scroll, перенёс изReadme.mdчасть про режим скролла.const { document } = useDOM()на получениеdocumentизdocument.body, черезownerDocument.AppRoot.test.tsx– добавил больше тестов, по ходу дела отрефакторил весь файл.packages/playwright/vkui/index.css– заменил CSS селекторе "регулярку" с$на*, т.к.$предполагал, что класс с токеном всегда будет в конце.styles/common.css– удалил селектор.vkuiиз правила, которые задаютfont-family,colorи т.п., т.к. они и так устанавливаются дляbody.hooks/useObjectMemo.ts– заменил кастомное сравнение объектов наisEqual()из@vkontakte/vkjs.