fix: linear gradient color stop fix up spec#45969
fix: linear gradient color stop fix up spec#45969intergalacticspacehighway wants to merge 9 commits into
Conversation
|
Going to mostly trust this based off of the unit tests. Function notation parsing is one of the first things on my list, once I get started back on that again. I suspect when I turned on “ReactNativeFeatureFlags.enableBackgroundStyleApplicator, I broke gradients on Android, since the new path is on the composite drawbable. I haven’t looked around at fixing that up yet. I think it might look like “BackgroundStyleApplicator.setBackgroundImage()”. |
yeah i agree to move everything to cpp parser eventually. I built a tiny parser in JS for this syntax instead of using regex. But this is working fine for now.
yes, it has affected the borders. Will take a look and do a PR by this weekend |
|
@NickGerleman has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
|
@NickGerleman merged this pull request in 1a49892. |
|
This pull request was successfully merged by @intergalacticspacehighway in 1a49892 When will my fix make it into a release? | How to file a pick request? |
Summary:
positiontopositionsin object style API. Optional string array here makes more sense. We'll add number array support oncepxsupport is added. Will do it as a follow up to this PR.TODOs: transition hint syntax support
linear-gradient(red, 50%, green)(Done locally, dependent on this PR).pxsupport.Changelog:
[GENERAL] [FIXED] - Linear gradient color stop spec.
Test Plan: