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

Issue with css.defineVars in React Native when using react-strict-dom in monorepo #260

Open
christianbach opened this issue Jan 18, 2025 · 5 comments
Labels
bug Something isn't working

Comments

@christianbach
Copy link

christianbach commented Jan 18, 2025

Describe the issue

We are using react-strict-dom in our React web app and React Native app, and everything works great in the web app. However, we are encountering issues in the React Native app when using css.defineVars from our styling package. Interestingly, the issue seems to be resolved when using ...css.props([]) instead of style=[].

We are bundling the app with Metro and have applied unstable_enablePackageExports to resolve potential export issues.

App Structure:

  • Packages Folder:

    • styling-generator: Imports Figma tokens and generates Stylex files for colors, typography, spacings, etc.
    • ui: Contains components used by both React and React Native apps.
  • Apps Folder:

    • Web: React web app.
    • React Native: React Native app.

Example (Simplified):

spacings.stylex.ts (exported from ui-generated):

import {css} from 'react-strict-dom';

export const spacing = css.defineVars({
  s8: 8,
  s16: 16,
  s32: 32,
});

Gap.tsx (exported from ui):

import {memo} from 'react';
import {css, html} from 'react-strict-dom';

import { spacing } from 'ui-generated/spacings.stylex';

const styles = css.create({
  small: {
    height: spacing.s8,
  },
  medium: {
    height: spacing.s16,
  },
  large: {
    height: spacing.s32,
  },
});

type Props = {
  size?: 'small' | 'medium' | 'large';
};

export const Gap = memo(({size}: Props) => {
  return <html.div style={[size && styles[size]]} />;
});

Issue:

  • In React Native, the styles defined using css.create() and passed through style=[] are not applied as expected (the height is undefined). Even though it works correctly in the web app.
  • Interestingly, if we use ...css.props([]) instead of style=[], the issue is resolved in React Native.
  • We are also seeing the following warning in the console:
    [warn] React Strict DOM: unrecognized custom property "--s16__id__202". This indicates that a custom CSS property is being used, but it's not recognized in React Strict DOM.

Any insight into why style=[] behaves differently in React Native would be greatly appreciated!

Expected behavior

  • Styles created with css.create that use vars should work correctly in both React and React Native when passed to style=[].

Steps to reproduce

"react": "18.3.1",
"react-native": "0.75.4",
"react-strict-dom": "0.0.33"
  1. Set up the project as described, with react-strict-dom in both the React and React Native apps.
  2. Use css.create to generate styles in the Gap component (as shown in the example).
  3. Observe the issue in the React Native app, where the styles are not applied correctly when using style=[].
  4. Replace style=[] with ...css.props([]) and verify that the issue is resolved in React Native.

Test case

No response

Additional comments

  • The issue appears only in React Native, and everything works as expected on the web.
  • The styling generator relies on Figma tokens, and stylex files are shared across both platforms.
  • The mono repo structure includes separate packages for the styling generator and UI components.
  • We are bundling with Metro and have applied unstable_enablePackageExports to resolve potential export issues.
@christianbach christianbach added the bug Something isn't working label Jan 18, 2025
@christianbach christianbach changed the title Issue with css.defineVars in React Native when using react-strict-dom in mono repo Issue with css.defineVars in React Native when using react-strict-dom in monorepo Jan 18, 2025
@necolas
Copy link
Contributor

necolas commented Jan 18, 2025

What does "not correctly" mean? I don't see any description of what the problem is. css.props is not part of the API in RSD and doesn't work with its elements

@christianbach
Copy link
Author

What does "not correctly" mean? I don't see any description of what the problem is. css.props is not part of the API in RSD and doesn't work with its elements

@necolas I'm sorry, it got lost in one of my edits. I updated the issue. But the problem we are facing is that the style of the Gap component resolves to {height: {}} in our react-native app.

@necolas
Copy link
Contributor

necolas commented Jan 18, 2025

We use this API without issue at Meta. Your example code has various syntax errors, so I can't tell what problem you might be experiencing. Can you create a test case that reproduces the issue using Expo's snack UI, then share the link?

@christianbach
Copy link
Author

Thanks for the response! I wanted to share some additional details:

In our setup, I got everything working if react-native imports all components, tokens, and themes exclusively from the ui-package, which consumes styling-generator and exports the stylex tokens.

However, when we mix imports directly from both ui-package and styling-generator, and both use defineVars, we encounter issues specifically in React Native, resulting in missing styles.

While debugging, I noticed that not all variables are present in the __customProperties) object . This seems to happen when multiple packages independently call defineVars.

I’ll work on creating a Snack later this week to better demonstrate the issue. Let me know if you have any advice or need more details in the meantime!

@necolas
Copy link
Contributor

necolas commented Jan 19, 2025

Thanks, that's helpful. I better understand the issue. We don't have a package setup like this internally, so it's the kind of thing that could not be working correctly without us noticing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants