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.

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(web,novui): initial implementation of var autocomplete in controls #6097

Merged
merged 46 commits into from
Jul 31, 2024

Conversation

ainouzgali
Copy link
Contributor

@ainouzgali ainouzgali commented Jul 17, 2024

What changed? Why was the change needed?

Screenshots

Screenshot 2024-07-22 at 18 38 34 Screenshot 2024-07-22 at 18 39 20
Expand for optional sections

Related enterprise PR

Special notes for your reviewer

@ainouzgali ainouzgali requested a review from a team July 17, 2024 15:30
Copy link

linear bot commented Jul 17, 2024

@ainouzgali ainouzgali changed the base branch from next to permissive-html-sanitization July 18, 2024 16:18
@ainouzgali
Copy link
Contributor Author

@antonjoel82 thanks for the style fixes! It seems to work fine now in web. Will appreciate your final overview and approval so we can merge this 💪

Copy link
Contributor

@antonjoel82 antonjoel82 left a comment

Choose a reason for hiding this comment

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

Awesome work!

},
},
},
staticCss: ['*'],
Copy link
Contributor

Choose a reason for hiding this comment

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

🗒 note (non-blocking): This tells Panda to generate styles for all variants and slots, regardless of which ones are actually used in code. When using defineSlotRecipe, Panda normally only generates classes and styles for recipes that the static checker detects are used in code.‏

.source Outdated Show resolved Hide resolved
@ainouzgali ainouzgali merged commit 43805d4 into next Jul 31, 2024
30 checks passed
@ainouzgali ainouzgali deleted the nv-4097-controls-auto-complete-in-the-dashboard branch July 31, 2024 07:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants