Skip to content

Add Fabric Component Props under API and Separate Old/New Components Section #1024

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

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

anupriya13
Copy link

@anupriya13 anupriya13 commented Apr 18, 2025

Description

  • Separates Old and New Arch Components Sections
  • Adds Fabric Component Props under New API for Windows.
    It doesn't contain Android / IOS specific props.

Why

What is the motivation for this change? Add a few sentences describing the context and overall goals of the pull request's commits.
Improve documentation

Resolves #1025

Refer https://github.com/facebook/react-native-website/blob/main/docs/image.md?plain=1

Screenshots

Add any relevant screen captures here from before or after your changes.
Before:

Recording.2025-04-19.234032.mp4

After: Added a new section which was missing in before "Windows Components - New Architecture"

website-new-vid.mp4
Microsoft Reviewers: Open in CodeFlow

@anupriya13 anupriya13 requested a review from a team as a code owner April 18, 2025 08:16
@anupriya13
Copy link
Author

@microsoft-github-policy-service agree company="Microsoft"

@anupriya13 anupriya13 added bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request website Issues with the website infrastructure labels Apr 18, 2025
@anupriya13 anupriya13 changed the title [0.79] Add Image Component Props under API [0.79] Add Image Component Props under API and Separate Old/New Components Section Apr 18, 2025
@anupriya13 anupriya13 changed the title [0.79] Add Image Component Props under API and Separate Old/New Components Section [0.79] [Draft] Add Fabric Component Props under API and Separate Old/New Components Section Apr 18, 2025
@Yajur-Grover
Copy link
Contributor

Couple of small things:

  • In the sidebar, the titles of the Old and New Arch sections could be changed to just be 'Components - New Architecture' and 'Components - Old Architecture'. The 'Windows' part is not necessary since it's the React Native for Windows website.
  • Some of the components that you have put in the New Architecture section are also present in Old Architecture, which may cause some confusion for newer users of React Native Windows. Maybe it might make sense to have a list of Core components that are present in (or planned to be present in) both Old and New Architecture, and then a section for Old Arch only components, and one for New Arch only components.

@@ -17,6 +17,7 @@
!versioned_docs/version-0.76/native-api/*-api-windows*.md
!versioned_docs/version-0.77/native-api/*-api-windows*.md
!versioned_docs/version-0.78/native-api/*-api-windows*.md
!versioned_docs/version-0.79/native-api/*-api-windows*.md
Copy link
Contributor

@TatianaKapos TatianaKapos Apr 18, 2025

Choose a reason for hiding this comment

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

Ah this is good question! We don't want to cut these documents yet, RNW 0.79 isn't released yet. Instead, this should just be changed in the next documentation (main) and we will create the 0.79 version-docs as part of the release process

Copy link
Author

@anupriya13 anupriya13 Apr 19, 2025

Choose a reason for hiding this comment

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

I have removed them. Please review.

Copy link
Contributor

@TatianaKapos TatianaKapos left a comment

Choose a reason for hiding this comment

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

We aren't ready to cut the 0.79 documentation, any changes/added documentation should be in the docs folder

@anupriya13
Copy link
Author

anupriya13 commented Apr 19, 2025

Couple of small things:

  • In the sidebar, the titles of the Old and New Arch sections could be changed to just be 'Components - New Architecture' and 'Components - Old Architecture'. The 'Windows' part is not necessary since it's the React Native for Windows website.
  • Some of the components that you have put in the New Architecture section are also present in Old Architecture, which may cause some confusion for newer users of React Native Windows. Maybe it might make sense to have a list of Core components that are present in (or planned to be present in) both Old and New Architecture, and then a section for Old Arch only components, and one for New Arch only components.
  1. Sure, will update the titles, as this doc is for Mac as well hence will keep Windows.
  2. The reason I kept it as different is because each page of old and new would have different props, and when we want to depreciate the old one it would be easier. Helps us track exactly what props are implemented. Also, the old arch documentation is not complete, wouldn't want to add effort in updating that.
    Hence, keeping these 2 separate only for now. We can update in future after discussing with PMs.

@anupriya13 anupriya13 requested a review from TatianaKapos April 19, 2025 07:49
@anupriya13 anupriya13 changed the title [0.79] [Draft] Add Fabric Component Props under API and Separate Old/New Components Section Add Fabric Component Props under API and Separate Old/New Components Section Apr 19, 2025
@Yajur-Grover
Copy link
Contributor

  1. The reason I kept it as different is because each page of old and new would have different props, and when we want to depreciate the old one it would be easier. Helps us track exactly what props are implemented. Also, the old arch documentation is not complete, wouldn't want to add effort in updating that.
    Hence, keeping these 2 separate only for now. We can update in future after discussing with PMs.

Okay, maybe we can apply the suggestion from #1014 then, and mark each page depending on whether it is applicable to Old Arch, New Arch or both.

@anupriya13 anupriya13 linked an issue Apr 21, 2025 that may be closed by this pull request
Copy link
Contributor

@chiaramooney chiaramooney left a comment

Choose a reason for hiding this comment

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

Blocking until design decision made about if we want to document all components instead of Windows-only.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request Needs: Attention 👋 website Issues with the website infrastructure
Projects
None yet
4 participants