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

[change] Remove backgroundClip prefix workaround #2427

Closed

Conversation

SukkaW
Copy link

@SukkaW SukkaW commented Nov 16, 2022

The PR continues re-implement #2254 and now targets 0.19-dev.

robinweser/inline-style-prefixer#221 has been merged and released in 6.0.2, the PR bumps the version of inline-style-prefixer and removes the workaround for background-clip when prefixing.

necolas and others added 11 commits August 30, 2022 14:49
* Support React 18 concurrency and constraints.
* Add new render / hydrate functions.
* Remove uses of findNodeHandle.
* Expose ability to unmount an application once ran.

Fix #1529
Close #2330
Encourage use of existing array syntax for multiple styles. Eventually
we'll want to remove the compose() API to simplify the overall styling API.
And remove deprecated AppState.removeListener API

Fix #2378
Previously deprecated, and not supported in the React Native Fabric
architecture.

Close #1935
Animated should now work with compiled and extracted styles. The
original styles are passed to components, rather than being flattened
into a new object that cannot be used by the style runtime to either
lookup the results of StyleSheet.create calls or consume extracted
styles. Inline styles that use AnimatedValue are moved into a seperate
object that is appended to the original styles.

Fix #2387
The resizeMode and tintColor props should be used instead. The styles
will be removed in a future release.

Fix #2383
These components have been replaced by Pressable and will be removed
from React Native.

Fix #2382
@codesandbox-ci
Copy link

codesandbox-ci bot commented Nov 16, 2022

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 ed2ca6e:

Sandbox Source
react-native-web-examples Configuration

@SukkaW SukkaW marked this pull request as draft November 16, 2022 11:21
@SukkaW
Copy link
Author

SukkaW commented Nov 16, 2022

Blocked by robinweser/inline-style-prefixer#225

@azigler
Copy link

azigler commented Nov 16, 2022

@SukkaW The blocking PR was merged this morning, and now react-native-web isn't working unless the version is rolled back. Is this ready to merge?

@robinweser
Copy link
Contributor

You can update to 7.0.0 to fix the issue here. Sorry for the inconvenience.

@captDaylight
Copy link

@SukkaW @azigler not ready until version bumped to 7.0.0 per @robinweser

@dutradotdev
Copy link

Hey @SukkaW
I am facing the issue Module not found: Error: Can't resolve 'inline-style-prefixer/lib/plugins/backgroundClip'
Is this issue related to your PR?

tks

@SukkaW
Copy link
Author

SukkaW commented Nov 17, 2022

You can update to 7.0.0 to fix the issue here. Sorry for the inconvenience.

@robinweser Looks like the fix should be released as 6.0.4, not 7.0.0 (otherwise the backport fix will not be applied to the existing react-native-web project).

@robinweser
Copy link
Contributor

@SukkaW I released a 6.0.4 that fixes the error with a 7.0.0 that cleans it up.

@necolas necolas deleted the branch necolas:0.19-dev March 28, 2023 16:59
@necolas necolas closed this Mar 28, 2023
@rosen91
Copy link

rosen91 commented Jul 2, 2023

@necolas I ran into this issue now, this fix was never published?
Using:
"name": "react-native-web",
"version": "0.19.6",

With Expo.

package-json still says "inline-style-prefixer": "^6.0.1"
I'm running it in a mono-repo if that has anything to say.

ERROR in ./node_modules/react-native-web/dist/modules/prefixStyles/static.js:1
Module not found: Can't resolve 'inline-style-prefixer/lib/plugins/backgroundClip'

1 | import backgroundClip from 'inline-style-prefixer/lib/plugins/backgroundClip';
2 | import crossFade from 'inline-style-prefixer/lib/plugins/crossFade';
3 | import cursor from 'inline-style-prefixer/lib/plugins/cursor';
4 | import filter from 'inline-style-prefixer/lib/plugins/filter';

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.

8 participants