Skip to content

Conversation

@acoates-ms
Copy link
Contributor

@acoates-ms acoates-ms commented Jul 8, 2024

Description

When changing border radius from non-zero to zero the clipping path and relative size adjustment do not get reset to null / 0, causing an incorrect border render.

Microsoft Reviewers: Open in CodeFlow

@acoates-ms acoates-ms requested a review from a team as a code owner July 8, 2024 20:03
@acoates-ms acoates-ms merged commit 34dc401 into microsoft:main Jul 9, 2024
@acoates-ms acoates-ms deleted the borderradiuschange branch July 9, 2024 19:53
acoates-ms added a commit to acoates-ms/react-native-windows that referenced this pull request Jul 10, 2024
* Handle changes to borderradius

* Change files

* format
@acoates-ms acoates-ms mentioned this pull request Jul 10, 2024
acoates-ms added a commit that referenced this pull request Jul 10, 2024
* [Fabric] [Win32] Add image response APIs to win32 exports (#13428)

* Add image response APIs to win32 exports

* Change files

* fix

* [Fabric] Handle changes to BorderRadius (#13422)

* Handle changes to borderradius

* Change files

* format

* fix change files
@jonthysell jonthysell added the Area: Fabric Support Facebook Fabric label Jul 15, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot added the New Architecture Broad category for issues that apply to the RN "new" architecture of Turbo Modules + Fabric label Jul 15, 2024
acoates-ms added a commit to acoates-ms/react-native-windows that referenced this pull request Jul 16, 2024
* Handle changes to borderradius

* Change files

* format
acoates-ms added a commit that referenced this pull request Aug 1, 2024
* [Fabric] Patch yoga to include changes to correctly recalculate layout on scale change (#13407)

* Patch yoga to handle dynamic scale changes

* Change files

* Update vnext/overrides.json

Co-authored-by: Marlene Cota <1422161+marlenecota@users.noreply.github.com>

* Update vnext/overrides.json

Co-authored-by: Marlene Cota <1422161+marlenecota@users.noreply.github.com>

* Update vnext/overrides.json

Co-authored-by: Marlene Cota <1422161+marlenecota@users.noreply.github.com>

* Update vnext/overrides.json

Co-authored-by: Marlene Cota <1422161+marlenecota@users.noreply.github.com>

* Update vnext/overrides.json

Co-authored-by: Marlene Cota <1422161+marlenecota@users.noreply.github.com>

* Update vnext/ReactCommon/TEMP_UntilReactCommonUpdate/yoga/yoga/config/Config.cpp

Co-authored-by: Marlene Cota <1422161+marlenecota@users.noreply.github.com>

* Update vnext/ReactCommon/TEMP_UntilReactCommonUpdate/yoga/yoga/config/Config.h

Co-authored-by: Marlene Cota <1422161+marlenecota@users.noreply.github.com>

* Update vnext/ReactCommon/TEMP_UntilReactCommonUpdate/yoga/yoga/config/Config.h

Co-authored-by: Marlene Cota <1422161+marlenecota@users.noreply.github.com>

* Update vnext/ReactCommon/TEMP_UntilReactCommonUpdate/yoga/yoga/node/LayoutResults.cpp

Co-authored-by: Marlene Cota <1422161+marlenecota@users.noreply.github.com>

* Update vnext/ReactCommon/TEMP_UntilReactCommonUpdate/yoga/yoga/node/LayoutResults.h

Co-authored-by: Marlene Cota <1422161+marlenecota@users.noreply.github.com>

* review feedback from yoga PR

---------

Co-authored-by: Marlene Cota <1422161+marlenecota@users.noreply.github.com>

* [Fabric] Handle scale factor changes (#13406)

* [Fabric] Handle scalefactor changes

* Change files

* [Fabric] Handle changes to BorderRadius (#13422)

* Handle changes to borderradius

* Change files

* format

* [Fabric] [Win32] Add image response APIs to win32 exports (#13428)

* Add image response APIs to win32 exports

* Change files

* fix

* [Fabric] Move to WinAppSDK types for KeyStatus, and ensure usages account for locked flag (#13435)

* [Fabric] Move to WinAppSDK types for KeyStatus, and ensure usages account for locked flag

* Change files

* Use scrollEnabled flag to enable/disable the ScrollView (#13427)

* Fix scrollview

* Change files

* Address feedback

* Return false for scroll event handlers when scroll is disabled

* Update scrollbar color based on scrollEnabled

* Rename OnThemeChanged to UpdateColorForScrollBarRegions

* [Fabric] Fix image component reference cycle (#13440)

* [Fabric] Fix image component reference cycle

* Change files

* format

* disable aggressive component deleted assert

* Use weak_ref for image didReceiveImage callback

* typo

* Unsubscribe from imageresponseobserver when deleted - aligns with core

* format

* [Fabric] call reportMount to implement UIManagerMountHook support (#13443)

* [Fabric] call reportMount to implement UIManagerMountHook support

* Change files

* Resolve Transform Matrix before Animation (#13450)

* Resolve transform before animation

* Change files

* Minor naming fix

* noexcept

* [Fabric] Text renders borders twice (#13445)

* [Fabric] Text renders borders twice

* Change files

* Add fabric test for text borders

* snapshot updates

* snapshots

* align overrides for 0.75 branch

* snapshot

---------

Co-authored-by: Marlene Cota <1422161+marlenecota@users.noreply.github.com>
Co-authored-by: Sharath Manchala <10109130+sharath2727@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Fabric Support Facebook Fabric New Architecture Broad category for issues that apply to the RN "new" architecture of Turbo Modules + Fabric

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants