Skip to content

Modify figma api changes#48

Open
Ho-Gyu-Lee wants to merge 1 commit intosimonoliver:mainfrom
Ho-Gyu-Lee:main
Open

Modify figma api changes#48
Ho-Gyu-Lee wants to merge 1 commit intosimonoliver:mainfrom
Ho-Gyu-Lee:main

Conversation

@Ho-Gyu-Lee
Copy link

No description provided.

@simonoliver
Copy link
Owner

Hi,
Thanks for the contribution. Would you be able to confirm your changes here? From looking at the commit I can see:

  • Added "role" attribute to FIGMA file data structure
  • Added a custom JSON converter to parse relativeTransform into a 2D array
  • Added SPACE_BETWEEN alignment option
  • Process all fills rather than just the first.

Is that right?

Also the current FigmaImage component doesn't support more than one fill though - did you find that your change gave you additional support that you needed?

Thanks,
Simon

@Ho-Gyu-Lee
Copy link
Author

Hello.
When I checked, I checked that the fills second had an image value. So, when I applied the first one, the image output was not done properly. I'm not good at English.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants