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.

Sign up for GitHub

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

Jump to bottom

feat: figma plugin for palette local variables #2870

Merged
merged 1 commit into from
Feb 28, 2025
Merged

feat: figma plugin for palette local variables #2870

merged 1 commit into from
Feb 28, 2025
+1,879 −539

Conversation

vscaiceanu-1a
Copy link
Member

Proposed change

First version of the Figma plugin for storing palettes in local variables

@vscaiceanu-1a vscaiceanu-1a requested a review from a team as a code owner February 19, 2025 06:44
@vscaiceanu-1a vscaiceanu-1a force-pushed the feature/figma-palette-plugin branch 2 times, most recently from 0bb818b to d1f8402 Compare February 19, 2025 06:53
@nx-cloud Nx Cloud
Copy link

nx-cloud bot commented Feb 19, 2025 •
edited
Loading

View your CI Pipeline Execution ↗ for commit 62f985d.

Command Status Duration Result
nx run-many --target=build --projects=eslint-pl... ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2025-02-28 10:11:37 UTC

@vscaiceanu-1a vscaiceanu-1a force-pushed the feature/figma-palette-plugin branch from d1f8402 to 0ffe36d Compare February 19, 2025 07:37
kpanot
kpanot reviewed Feb 19, 2025
View reviewed changes
apps/palette-generator/manifest.json
@@ -0,0 +1,18 @@
{
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do they provide a Json Schema?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I couldn't find one.
There's an outdated one here https://github.com/figmatic-team/figma-plugin-manifest
and a request for one here https://forum.figma.com/suggest-a-feature-11/jsonschema-for-manifest-json-21692

@vscaiceanu-1a vscaiceanu-1a force-pushed the feature/figma-palette-plugin branch 3 times, most recently from cac88b4 to a28da7d Compare February 19, 2025 08:24
@codecov Codecov
Copy link

codecov bot commented Feb 19, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.76%. Comparing base (a902bec) to head (62f985d).
Report is 2 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

matthieu-crouzet
matthieu-crouzet reviewed Feb 19, 2025
View reviewed changes
@vscaiceanu-1a vscaiceanu-1a force-pushed the feature/figma-palette-plugin branch 5 times, most recently from 611f047 to 46a42ef Compare February 19, 2025 14:07
@kpanot kpanot mentioned this pull request Feb 20, 2025
Open
10 tasks
@vscaiceanu-1a vscaiceanu-1a force-pushed the feature/figma-palette-plugin branch 2 times, most recently from 939b4a6 to f5b4c8e Compare February 20, 2025 15:58
@vscaiceanu-1a vscaiceanu-1a force-pushed the feature/figma-palette-plugin branch from f5b4c8e to e4c2dd2 Compare February 20, 2025 16:38
@vscaiceanu-1a vscaiceanu-1a force-pushed the feature/figma-palette-plugin branch from 0c98a61 to 70ffd1c Compare February 24, 2025 08:54
@vscaiceanu-1a vscaiceanu-1a force-pushed the feature/figma-palette-plugin branch 2 times, most recently from 9a90f64 to ccd6869 Compare February 24, 2025 15:09
@vscaiceanu-1a vscaiceanu-1a force-pushed the feature/figma-palette-plugin branch from ccd6869 to 8a13e0c Compare February 24, 2025 16:59
@vscaiceanu-1a vscaiceanu-1a force-pushed the feature/figma-palette-plugin branch from 8a13e0c to 32ffad9 Compare February 25, 2025 08:25
kpanot
kpanot reviewed Feb 25, 2025
View reviewed changes
apps/palette-generator/tsconfig.build.plugin.json
@@ -0,0 +1,19 @@
// IDE usage only
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it only used by the IDE?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, I'm transpiling the plugin part with esbuild and I don't need a tsconfig file.

fpaul-1A
fpaul-1A previously approved these changes Feb 25, 2025
View reviewed changes
@vscaiceanu-1a vscaiceanu-1a force-pushed the feature/figma-palette-plugin branch 2 times, most recently from 98709bd to 2c914b4 Compare February 26, 2025 16:20
fpaul-1A
fpaul-1A previously approved these changes Feb 27, 2025
View reviewed changes
@vscaiceanu-1a vscaiceanu-1a force-pushed the feature/figma-palette-plugin branch from 2c914b4 to 8380504 Compare February 27, 2025 16:08
@vscaiceanu-1a vscaiceanu-1a force-pushed the feature/figma-palette-plugin branch from 8380504 to 62f985d Compare February 27, 2025 16:34
@vscaiceanu-1a vscaiceanu-1a added this pull request to the merge queue Feb 28, 2025
Merged via the queue into main with commit 9556281 Feb 28, 2025
45 of 46 checks passed
@vscaiceanu-1a vscaiceanu-1a deleted the feature/figma-palette-plugin branch February 28, 2025 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kpanot kpanot kpanot left review comments

@matthieu-crouzet matthieu-crouzet matthieu-crouzet left review comments

@fpaul-1A fpaul-1A fpaul-1A approved these changes

@cpaulve-1A cpaulve-1A cpaulve-1A approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request project:@ama-terasu/cli project:cascading-gh-action project:get-npm-tag-gh-action project:new-version-gh-action project:@o3r/core project:@o3r/eslint-config project:@o3r/eslint-config-otter project:@o3r/palette-generator project:release-gh-action
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@vscaiceanu-1a @kpanot @matthieu-crouzet @fpaul-1A @cpaulve-1A