Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[0.74] [Fabric] Port various fabric fixes #13454

Merged
merged 7 commits into from
Jul 16, 2024

Conversation

acoates-ms
Copy link
Contributor

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

Description

Port these PRs to 0.74
#13435
#13427
#13440
#13443
#13450
#13445

Microsoft Reviewers: Open in CodeFlow

acoates-ms and others added 7 commits July 16, 2024 13:41
…ount for locked flag (microsoft#13435)

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

* Change files
)

* 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

* 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
…crosoft#13443)

* [Fabric] call reportMount to implement UIManagerMountHook support

* Change files
* Resolve transform before animation

* Change files

* Minor naming fix

* noexcept
* [Fabric] Text renders borders twice

* Change files

* Add fabric test for text borders

* snapshot updates

* snapshots
@acoates-ms acoates-ms requested review from a team as code owners July 16, 2024 20:48
@acoates-ms acoates-ms enabled auto-merge (squash) July 16, 2024 21:46
@acoates-ms acoates-ms merged commit 7d828ea into microsoft:0.74-stable Jul 16, 2024
95 checks passed
@acoates-ms acoates-ms deleted the fabricfixes branch July 16, 2024 23:01
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

4 participants