Skip to content

Conversation

@chrisglein
Copy link
Member

@chrisglein chrisglein commented Feb 9, 2023

Description

Backporting #10969

0.71 brings in new support for rowGap, columnGap and gap flex properties. This hooks up the new properties.

Type of Change

  • New feature (non-breaking change which adds functionality)

Why

Feature was added in core 0.71 but the RNW change didn't make 0.71.0

Testing

Original verification

Verified the new gap test on the ViewExample test page in RNTester

New verification

Patched files into an app that uses gap and ensured layout was modified.
Ran in RNTester to verify gap test:
image

Microsoft Reviewers: Open in CodeFlow

acoates-ms and others added 2 commits February 9, 2023 10:03
* Implement rowGap, columnGap and gap flex properties

* Change files
@chrisglein chrisglein requested a review from a team as a code owner February 9, 2023 18:07
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.

👍

@chrisglein chrisglein requested a review from acoates-ms February 9, 2023 19:38
@chrisglein
Copy link
Member Author

@acoates-ms Speak up if there's any reason not to backport this

@acoates-ms acoates-ms merged commit cfef068 into microsoft:0.71-stable Feb 9, 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.

4 participants