-
Notifications
You must be signed in to change notification settings - Fork 930
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
chore: Unreleased bugs with advanced section #4882
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
apps/builder/app/builder/features/style-panel/sections/advanced/parse-style-input.ts
Show resolved
Hide resolved
@johnsicili how exactly did you manage to get it to insert invalid property name? |
Yeah, you either select one of the shortcuts or you escape and insert what you wrote. Should I add to the dropdown additionally the same shorthand notation user has so they understand they don't have to choose the shorthand provided? |
I know forsure it will add via typing/pasting full declaration |
Yes. I did not think of hitting escape but even with knowing that, it requires an unnecessary step so adding to the autocomplete is a good idea |
unset2.mp4 |
Description
https://github.com/user-attachments/assets/a2446685-e7dc-4c01-bf03-4ee64b8f5e16
Steps for reproduction
Code Review
Before requesting a review
Before merging
.env
file