Skip to content

Split RUIProvider into two#583

Merged
mbohal merged 1 commit into
masterfrom
bc/split-ruiprovider
Jan 23, 2025
Merged

Split RUIProvider into two#583
mbohal merged 1 commit into
masterfrom
bc/split-ruiprovider

Conversation

@mbohal

@mbohal mbohal commented Dec 20, 2024

Copy link
Copy Markdown
Contributor

This commit splits RUIProvider into TranslationsProvider and GlobalPropsProvider.

The reasons fo this change are:

  • Improve performance by reducing rerenders in cases when the context value change dynamically.

  • Make code more readable by splitting isolated functionality into isolated prividers.

Comment thread src/providers/translations/__tests__/TranslationsProvider.test.jsx Outdated
Comment thread src/providers/translations/TranslationsProvider.jsx Outdated
Comment thread src/providers/globalProps/withGlobalProps.jsx
Comment thread src/docs/customize/translations.md Outdated
@mbohal
mbohal force-pushed the bc/split-ruiprovider branch 3 times, most recently from f431bc0 to 226a3a0 Compare January 21, 2025 15:56

@adamkudrna adamkudrna left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

Comment thread src/components/Alert/README.md
Comment thread src/components/Badge/Badge.jsx
Comment thread src/components/InputGroup/InputGroup.jsx
Comment thread src/docs/customize/global-props.md
Comment thread src/index.js Outdated
Comment thread src/docs/customize/translations.md Outdated
Comment thread src/providers/globalProps/index.js Outdated
Comment thread src/providers/translations/__tests__/TranslationsProvider.test.jsx Outdated
Comment thread src/providers/translations/index.js Outdated
Comment thread src/providers/translations/__tests__/TranslationsProvider.test.jsx
This commit splits `RUIProvider` into `TranslationsProvider` and
`GlobalPropsProvider`.

The reasons fo this change are:

* Improve performance by reducing rerenders in cases when the context
  value change dynamically.

* Make code more readable by splitting isolated functionality into
  isolated prividers.
@mbohal
mbohal force-pushed the bc/split-ruiprovider branch from 226a3a0 to baf0d2a Compare January 23, 2025 07:36
@mbohal
mbohal merged commit ca1c36a into master Jan 23, 2025
@mbohal
mbohal deleted the bc/split-ruiprovider branch January 23, 2025 07:39
@adamkudrna adamkudrna linked an issue Jan 24, 2025 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Consider spliting RUIProvider and exporting HOCs

3 participants