-
-
Notifications
You must be signed in to change notification settings - Fork 98
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
feat(app): allow zero padding #625
feat(app): allow zero padding #625
Conversation
Review or Edit in CodeSandboxOpen the branch in Web Editor • VS Code • Insiders |
🦋 Changeset detectedLatest commit: 29df7d7 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
1905d91
to
4a3940c
Compare
apps/codeimage/src/components/PropertyEditor/FrameStyleForm.tsx
Outdated
Show resolved
Hide resolved
apps/codeimage/src/components/PropertyEditor/FrameStyleForm.tsx
Outdated
Show resolved
Hide resolved
Thanks @riccardoperra for the feedback, I will look into it |
@riccardoperra When you get a chance, can you please check my replies and let me know your thoughts |
4a3940c
to
29df7d7
Compare
@riccardoperra New changes are ready to review. Please review when you get a chance. |
Hi @SarjuHansaliya, the code is ok but I miss some time currently. I'll merge it but I'll deploy together with other features in a tbd date. I appreciate your help 😄 |
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Issue Number: #623
What is the new behavior?
Does this PR introduce a breaking change?