Skip to content
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

Initial transform re-applied when View is moved #12819

Open
rozele opened this issue Mar 11, 2024 · 1 comment
Open

Initial transform re-applied when View is moved #12819

rozele opened this issue Mar 11, 2024 · 1 comment
Labels
Area: View bug Old Architecture Broad category for issues that apply to the RN "old" architecture of Cxx Modules + Paper Partner: Facebook
Milestone

Comments

@rozele
Copy link
Collaborator

rozele commented Mar 11, 2024

Problem Description

XAML fires the Loaded event every time you "move" a keyed component in react-native. The reason is that you remove the XAML view from the parent's collection of children then re-add it at the new index.

Steps To Reproduce

https://snack.expo.dev/yItxPvjSwp7gqhNTAjk0s

Expected Results

Windows behavior should match the Web behavior in the Expo Snack (where after unsetting the scale transform, the unset transform behavior persists).

However, because the Loaded callback in FrameworkElementViewManager is never cleared, the transform gets re-applied :(

CLI version

npx react-native -v

Environment

npx react-native info

Target Platform Version

None

Target Device(s)

No response

Visual Studio Version

None

Build Configuration

None

Snack, code example, screenshot, or link to a repository

No response

@rozele rozele added the bug label Mar 11, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot added Needs: Triage 🔍 New issue that needs to be reviewed by the issue management team (label applied by bot) Partner: Facebook labels Mar 11, 2024
@chrisglein chrisglein added Area: View Old Architecture Broad category for issues that apply to the RN "old" architecture of Cxx Modules + Paper and removed Needs: Triage 🔍 New issue that needs to be reviewed by the issue management team (label applied by bot) labels Mar 14, 2024
@chrisglein
Copy link
Member

Interesting issue. As this is on old architecture, we wouldn't prioritize making a change here but if there's a PR with a fix we'd look at it.

@chrisglein chrisglein added this to the Backlog milestone Mar 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: View bug Old Architecture Broad category for issues that apply to the RN "old" architecture of Cxx Modules + Paper Partner: Facebook
Projects
None yet
Development

No branches or pull requests

2 participants