microsoft / microsoft/react-native-windows
Unfork AnimatedPropSerializer.cpp override added during 0.85 nightly integration
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- C++
- Sterne
- 17.3k
- Forks
- 1.2k
- Ø Merge
- 1 T. 13 Std.
- Gemergte PRs (30 T.)
- 33
Beschreibung
Problem Description
During the integration of RN 0.85.0-nightly-20260128-36f07a1b2 (#16245), the upstream AnimatedPropSerializer.cpp uses Color as a plain int32_t, but on Windows Color is a struct. This required a local patched copy with a helper function to convert Windows Color to int32_t ARGB format.
Current Fix
A local patched copy was created at vnext/ReactCommon/TEMP_UntilReactCommonUpdate/react/renderer/animationbackend/AnimatedPropSerializer.cpp with a colorToInt32() helper that calls color.AsWindowsColor().
Link to the integration PR where the error originated
#16245
Link to react-native commit
https://github.com/facebook/react-native/compare/f15985f4f...36f07a1b2
Forked files
No response
Upstream PR in react-native
No response
Build Configuration
None
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginnen Sie mit vnext/ReactCommon/TEMP_UntilReactCommonUpdate/react/renderer/animationbackend/AnimatedPropSerializer.cpp und untersuchen Sie die Hilfsfunktion colorToInt32() sowie ihre Windows-spezifische Konvertierung. Vergleichen Sie die Datei mit dem react-native-Commit, der in Integration PR #16245 verlinkt ist, und überprüfen Sie anschließend, dass die lokale gepatchte Kopie wieder aus dem Fork entfernt werden kann, sobald Upstream Windows Color unterstützt und der Windows-Integrations-Build erfolgreich ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- cpp, react-native
- Bereich
- desktop
- Issue-Typ
- Refactoring
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Ruhig
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 48/100