Skip to content

Conversation

acoates-ms
Copy link
Contributor

@acoates-ms acoates-ms commented Sep 10, 2021

This provides better type information for some of the additional properties that we've added to view in react-native-windows.

These properties are available by using import {ViewWindows} from 'react-native-windows' instead of import {View} from 'react-native'

Microsoft Reviewers: Open in CodeFlow

@acoates-ms acoates-ms requested a review from a team as a code owner September 10, 2021 21:15
@agniuks
Copy link
Contributor

agniuks commented Sep 10, 2021

@acoates-ms Not directly related to this PR, but I was curious if there might be a way to use this to provide info about windows-only inputs for RN-core props? -- for example currently we have to use @ts-ignore when setting accessibilityRole="list"

@acoates-ms
Copy link
Contributor Author

We should really update the official @types/react-native project, to include the windows properties.
https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/react-native/index.d.ts

@acoates-ms acoates-ms merged commit 5dbd74b into microsoft:main Sep 14, 2021
@acoates-ms acoates-ms deleted the bettertypes branch September 14, 2021 19:50
* See https://github.com/ReactWindows/discussions-and-proposals/blob/harinik-accessibility/proposals/0000-accessibilityapis-lists.md
*/
accessibilitySetSize?: number;
focusable?: boolean;
Copy link
Contributor

Choose a reason for hiding this comment

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

Did you remove this prop on purpose?

Copy link
Contributor

Choose a reason for hiding this comment

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

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.

3 participants