lidofinance / ui

React UI Components for Lido projects.
MIT License
20 stars 16 forks source link

fix: nested cookie theme provider #384

Closed solidovic closed 1 year ago

solidovic commented 1 year ago

Description

Fix bug with nested CookieThemeProvider.

Bug behavior

Flash rendering (about 1 second) with default theme (white theme) and after rendering with parent provider (CookieThemeProvider) theme

Fixed behavior

Rendering with parent provider (CookieThemeProvider) theme

Example case:

SomeWeb3Package.tsx

import { CookieThemeProvider } from '@lidofinance/lido-ui'
// others imports...

export const ProviderWeb3: FC = () => (
  <CookieThemeProvider>
    ...
  </CookieThemeProvider>
)

App.tsx

import { CookieThemeProvider } from '@lidofinance/lido-ui'
import { ProviderWeb3 } from 'SomeWeb3Package'
// others imports...

const App: FC = () => (
  <ProviderWeb3>
    <SomeProviders>
      <CookieThemeProvider>
        ...
      </CookieThemeProvider>
    </SomeProviders>
  </ProviderWeb3>
);