Skip to content

Add info about v-bind in CSS under CSSProperties #1518

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

Merged
merged 1 commit into from
Feb 12, 2022

Conversation

pleek91
Copy link

@pleek91 pleek91 commented Feb 11, 2022

When using SFC it's often not necessary to do type augmentation if v-bind in CSS is used.

Description of Problem

Adding custom properties in a template causes a type error because custom properties are not accounted for in CSSProperties. Searching how to fix that error frequently leads to the docs on CSSProperties and how to augment the type.

Proposed Solution

Using v-bind in CSS is often the better solution or is at least relevant so linking to that documentation will help convey another solution to fixing the type error.

When using SFC its often not necessary to do type augmentation if v-bind in CSS is used.
@netlify
Copy link

netlify bot commented Feb 11, 2022

✔️ Deploy Preview for vuejs ready!

🔨 Explore the source changes: 7ff8c24

🔍 Inspect the deploy log: https://app.netlify.com/sites/vuejs/deploys/620679e224f78c00085afc64

😎 Browse the preview: https://deploy-preview-1518--vuejs.netlify.app

Copy link
Member

@NataliaTepluhina NataliaTepluhina left a comment

Choose a reason for hiding this comment

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

@pleek91 thank you for adding this!

@NataliaTepluhina NataliaTepluhina merged commit 32754df into vuejs:main Feb 12, 2022
@pleek91 pleek91 deleted the patch-3 branch February 12, 2022 15:36
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.

2 participants