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

fix(runtime-dom): ensure customElement handles empty props correctly. (fix #6361,#6895) #6182

Merged
merged 1 commit into from Nov 1, 2022

Conversation

LinusBorg
Copy link
Member

@LinusBorg LinusBorg commented Jun 25, 2022

If a component wrapped with defineCustomElement() did not hae any props defined, and was passed any custom attribute (such as, in #6361, a data-v attribute), it would throw an error.

closes #6163, close #6895

@LinusBorg LinusBorg added 🔨 p3-minor-bug Priority 3: this fixes a bug, but is an edge case that only affects very specific usage. scope: custom elements labels Jun 25, 2022
@LinusBorg LinusBorg moved this from PRs to Ready to Merge in Custom Elements Bugsquash Jun 25, 2022
@LinusBorg LinusBorg moved this from Ready to Merge to PRs in Custom Elements Bugsquash Jun 25, 2022
@LinusBorg LinusBorg requested a review from antfu October 20, 2022 11:01
@LinusBorg LinusBorg added ❗ p4-important Priority 4: this fixes bugs that violate documented behavior, or significantly improves perf. and removed 🔨 p3-minor-bug Priority 3: this fixes a bug, but is an edge case that only affects very specific usage. labels Oct 20, 2022
@LinusBorg LinusBorg linked an issue Oct 20, 2022 that may be closed by this pull request
@LinusBorg LinusBorg changed the title fix(runtime-dom): ensure customElement handles empty props correctly. (fix #6361) fix(runtime-dom): ensure customElement handles empty props correctly. (fix #6361,#6895) Oct 20, 2022
fix Scoped attribute in Vue file affects the use of web component #6163,#6895
@LinusBorg LinusBorg force-pushed the linusborg/custom-element-handle-empty-props-6163 branch from e527c8c to 6d677d0 Compare October 20, 2022 11:08
@LinusBorg
Copy link
Member Author

@antfu small ping? 😅

@LinusBorg LinusBorg merged commit f67bb50 into main Nov 1, 2022
Custom Elements Bugsquash automation moved this from PRs to Done Nov 1, 2022
@LinusBorg LinusBorg deleted the linusborg/custom-element-handle-empty-props-6163 branch November 1, 2022 08:49
chrislone pushed a commit to chrislone/core that referenced this pull request Feb 4, 2023
zhangzhonghe pushed a commit to zhangzhonghe/core that referenced this pull request Apr 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
❗ p4-important Priority 4: this fixes bugs that violate documented behavior, or significantly improves perf. scope: custom elements
3 participants