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

Jump to bottom

fix(runtime-dom): style update error when component use shorthand properties #7425

Merged
merged 2 commits into from Jan 9, 2023
Merged

fix(runtime-dom): style update error when component use shorthand properties #7425

merged 2 commits into from Jan 9, 2023

Conversation

yurj26
Copy link
Contributor

@yurj26 yurj26 commented Dec 28, 2022

@zhangzhonghe zhangzhonghe self-requested a review December 29, 2022 07:53
zhangzhonghe
zhangzhonghe approved these changes Dec 29, 2022
View reviewed changes
Copy link
Member

@zhangzhonghe zhangzhonghe left a comment

Choose a reason for hiding this comment

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

LGTM

@yurj26
Copy link
Contributor Author

yurj26 commented Dec 29, 2022

LGTM

thanks

@yurj26
Copy link
Contributor Author

yurj26 commented Jan 3, 2023

LGTM

When will this PR be merged?

@xiaoxiangmoe
Copy link
Contributor

LGTM

@LinusBorg LinusBorg merged commit b7cfa6f into vuejs:main Jan 9, 2023
chrislone pushed a commit to chrislone/core that referenced this pull request Feb 4, 2023
@yurj26
fix(runtime-dom): style update error when component use shorthand pro…
a05825f 
…perties (vuejs#7425)

* fix(runtime-dom): style update error when component use shorthand properties

* test(runtime-dom): style update with shorthand properties
zhangzhonghe pushed a commit to zhangzhonghe/core that referenced this pull request Apr 12, 2023
@yurj26 @zhangzhonghe
fix(runtime-dom): style update error when component use shorthand pro…
5648575 
…perties (vuejs#7425)

* fix(runtime-dom): style update error when component use shorthand properties

* test(runtime-dom): style update with shorthand properties
@KaelWD KaelWD mentioned this pull request May 27, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Justineo Justineo Justineo approved these changes

@sxzz sxzz sxzz approved these changes

@zhangzhonghe zhangzhonghe Awaiting requested review from zhangzhonghe

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@yurj26 @xiaoxiangmoe @Justineo @sxzz @zhangzhonghe @LinusBorg