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

feat: add borderCurve and pointerEvents style properties #228

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

mlecoq
Copy link
Contributor

@mlecoq mlecoq commented Feb 9, 2023

Description

see comment #188 (comment)

Checklist

@mlecoq
Copy link
Contributor Author

mlecoq commented Feb 9, 2023

it needs this first facebook/react-native@a0800ff (types are now in react-native repo)

@@ -107,12 +111,12 @@ describe('createRestyleComponent', () => {
it('does not pass styling properties to the child', () => {
const {root} = render(
<ThemeProvider theme={theme}>
<Component opacity={0.5} pointerEvents="auto" />
<Component opacity={0.5} removeClippedSubviews />
Copy link
Contributor

Choose a reason for hiding this comment

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

Why was pointerEvents replaced by removeClippedSubviews?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Because pointerEvents is now present in style since rn 0.71 so this test failed. I have used removeClippedSubview but I could choose another property (that is not a style one)

Copy link
Contributor

@mattfrances mattfrances left a comment

Choose a reason for hiding this comment

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

React Native's borderCurve and pointerEvents don't seem to have been added to React Native's ViewStyle type. When clicking through ViewStyle on this line, borderCurve and pointerEvents are not included (see image below). This is causing the build failures. Do you know why they're not being included?

Screenshot 2023-02-10 at 11 27 32 AM

@mlecoq
Copy link
Contributor Author

mlecoq commented Feb 10, 2023

Yes, I added them in @types/react-native but now types are directly in react-native, a similar PR has been merged but related commits are not in react native 0.71.2. We have to wait a future version

@mlecoq
Copy link
Contributor Author

mlecoq commented Feb 10, 2023

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