Skip to content

Add cssInterop options to the Nativewind plugin#890

Merged
jbroma merged 2 commits intocallstack:mainfrom
ceopaludetto:feat/nativewind-css-options
Jan 22, 2025
Merged

Add cssInterop options to the Nativewind plugin#890
jbroma merged 2 commits intocallstack:mainfrom
ceopaludetto:feat/nativewind-css-options

Conversation

@ceopaludetto
Copy link
Copy Markdown
Contributor

@ceopaludetto ceopaludetto commented Jan 21, 2025

Summary

Introduces a new cssToReactNativeRuntimeOptions option (name suggestions are welcome) that forwards directly to cssToReactNativeRuntime allowing configurations like inlineRem and selectorPrefix

Test plan

  • testers work

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Jan 21, 2025

🦋 Changeset detected

Latest commit: a6af8a9

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@callstack/repack-plugin-nativewind Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link
Copy Markdown

vercel bot commented Jan 21, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
repack-website ⬜️ Ignored (Inspect) Visit Preview Jan 21, 2025 10:44pm

@ceopaludetto ceopaludetto force-pushed the feat/nativewind-css-options branch from 30d8b00 to 4380558 Compare January 21, 2025 20:06
@ceopaludetto ceopaludetto changed the title Feat/nativewind css options Add cssToReactNativeRuntime options to the Nativewind plugin Jan 21, 2025
Copy link
Copy Markdown
Contributor

@jbroma jbroma left a comment

Choose a reason for hiding this comment

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

thank you for your contribution @ceopaludetto, everything looks good overall.

One nit - I think we could make the prop name shorter like in the suggestions, WDYT?

Comment thread packages/plugin-nativewind/src/plugin.ts Outdated
Comment thread packages/plugin-nativewind/src/plugin.ts Outdated
Comment thread apps/tester-app/rspack.config.mjs Outdated
@ceopaludetto ceopaludetto force-pushed the feat/nativewind-css-options branch from 4380558 to 70c2f8c Compare January 21, 2025 22:43
@ceopaludetto ceopaludetto requested a review from jbroma January 21, 2025 22:43
@ceopaludetto ceopaludetto force-pushed the feat/nativewind-css-options branch from 70c2f8c to a6af8a9 Compare January 21, 2025 22:44
Copy link
Copy Markdown
Contributor

@jbroma jbroma left a comment

Choose a reason for hiding this comment

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

LGTM 🎉

@jbroma jbroma changed the title Add cssToReactNativeRuntime options to the Nativewind plugin Add cssInterop options to the Nativewind plugin Jan 22, 2025
@jbroma jbroma merged commit 361b7b9 into callstack:main Jan 22, 2025
@jbroma
Copy link
Copy Markdown
Contributor

jbroma commented Jan 22, 2025

Thank you again for your contribution @ceopaludetto 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants