When importing component inside custom element, style is discarded
Version
3.2.14
Reproduction link
Steps to reproduce
git clone git@github.com:gnuletik/vue-ce-import-comp-style.git
cd vue-ce-import-comp-style
yarn run dev
Open browser
What is expected?
CSS of OtherComponent.vue should be applied. The text "It should be blue" should be blue.
What is actually happening?
Style is not applied.
I tried renaming OtherComponent to OtherComponent.ce.vue, but the result is the same.
This is useful when writing a set of custom elements to have shared components between custom elements.
I just realized there is already an open PR about this : https://github.com/vuejs/vue-next/pull/4309
I see 👁️ . My changes are very similar, just its more recursive, meaning also components at any level are interested.
Ok, this should be safer now. This addition will add any child component style to the main parent, no matter how deeply nested it is
Anyone know if it will be merged? It blocks me from using Vue 3 :/ @raffobaffo What about third party nested components (e.g. imported from some package)? Will your solution also work for them?
We will solve this, it might take a few more days or weeks though.
@LinusBorg Ok, thanks for quick reply, it's good to know that it will be solved at some point in the future :)
@pawel-marciniak Yes, it is. @LinusBorg Great to hear. Atm to overcome this problem, we are using an own extended version of the defineCustomElement that provides that hack I inserted in the pr.
@raffobaffo would you mind sharing how did you extend it? I tried it here by basically cloning /runtime-dom/src/apiCustomElement.ts
with your changes but got a lot of syntax error from TS.
@lucasantunes-ciandt sorry but not TS for me on this project. This is pretty much the hack I built:
import { defineCustomElement as rootDefineCustomElement } from 'vue'
[...]
export const getChildrenComponentsStyles = (component) => {
let componentStyles = [];
if (component.components) {
componentStyles = Object.values(component.components).reduce(
(
aggregatedStyles,
nestedComponent,
) => {
if (nestedComponent?.components) {
aggregatedStyles = [
...aggregatedStyles,
...getChildrenComponentsStyles(nestedComponent),
];
}
return nestedComponent.styles
? [...aggregatedStyles, ...nestedComponent.styles]
: aggregatedStyles;
}, [],
);
}
if (component.styles) {
componentStyles.push(...component.styles);
}
return [...new Set(componentStyles)];
};
export const defineCustomElement = (component) => {
// Attach children styles to main element
// Should be removed once https://github.com/vuejs/vue-next/pull/4695
// gets merged
component.styles = getChildrenComponentsStyles(component);
const cElement = rootDefineCustomElement(component);
// Programmatically generate name for component tag
const componentName = kebabize(component.name.replace(/\.[^/.]+$/, "") );
// eslint-disable-next-line tree-shaking/no-side-effects-in-initialization
customElements.define(componentName, cElement);
// Here we are attaching a <ling ref="style" href="/style.css" ...
// This is to have the css outside of the shadow dom
// also available inside the shadow root without inlining them
// The browser will automatically use the available declaration and won't
// make multiple calls
const componentShadowDom = document.querySelector(componentName)?.shadowRoot;
if(componentShadowDom){
const styleLink = document.createElement('link');
styleLink.setAttribute('rel', 'stylesheet');
styleLink.setAttribute('href', 'style.css');
componentShadowDom.appendChild(styleLink);
}
}
@raffobaffo Thank you so much! I thought you were literally extending the whole file :sweat_smile:
I tried this code here, but unfortunately it doesn't work. I suppose it expects the components are all custom elements, e.g.
Component.ce.vue
and ChildComponent.ce.vue
.
When my components are named Component.vue
they don't come with any styles
prop, only when they're Component.ce.vue
.
I reckon we'll need to wait for your PR to be merged.
Oopsie, actually this works! Thank you @raffobaffo!!
I was a bit confused about the meaning of .ce.vue
, now I realize the extension is only a Custom Element MODE and not actually declaring those components as custom elements. This means I really should have them as .ce.vue
in order for this to work, even after this PR is merged.
So my two cents to the Issue Owner: OtherComponent.vue
should be OtherComponent.ce.vue
indeed.
@raffobaffo I had only to change the injection a little bit, since I may have multiple instances of the same Custom Element:
document.querySelectorAll(customElementName).forEach((element) => {
if (!element.shadowRoot) return;
const styleLink = document.createElement('link');
styleLink.setAttribute('rel', 'stylesheet');
styleLink.setAttribute('href', 'style.css');
element.shadowRoot.appendChild(styleLink);
});
@lucasantunes-ciandt Nice find 👍 ! Still this block is not part of the PR, because this actually make sense to have it in a separate module that is loaded just when custom elements are needed. In my case I have a components library and I want to have 2 exports, one to be consumed by Vue applications and one for browsers/html. This part would maybe make more sense in some Vite or VueCli plugin 🤔 .
@raffobaffo Exactly! In our case we're using Vue inside a CMS, so we'll be only exporting custom elements to use within it instead of creating a Vue app, so we'll definitely keep that part because we need some external styles and even scripts from the CMS to be injected into the elements.
@raffobaffo would your solution also inject styles into the shadow DOM from child components imported within the defineCustomElement
component that originate from external packages?
As an example, if I define and export a component with defineCustomElement
, but within that element is a button component that is being imported from an external package (and is not created with defineCustomElement
).
Also @lucasantunes-ciandt do you have a working example using the reproduction repo?
@adamlewkowicz hard to answer. Depends from how the imported element (third party) is structured. Does it comes with is own inline styles? If yes, it should work, if not, it cant.
@raffobaffo would you be willing to update your reproduction repo linked above with a new branch that utilizes your fix? I tried what you have but wasn't able to get it working for some reason. Would be much appreciated!
As a workaround I've created component, which tracks Mutation in document.head
and inline style tags (must includes comment /* VueCustomElementChildren */
) into shadow dom in dev mode, for production it inlines link for stylesheets:
CustomElements.vue
<script setup lang="ts">
/***
* @see https://github.com/vuejs/vue-next/issues/4662
**/
import { computed } from "vue";
import { onMounted, onUnmounted, ref } from "@vue/runtime-core";
const props = defineProps<{
cssHref?: string;
}>();
const inlineStyles = ref<HTMLElement[]>([]);
const html = computed(() =>
// @ts-ignore
import.meta.env.PROD && props.cssHref
? `<link href="${props.cssHref}" rel="stylesheet"/>`
: `<style>${inlineStyles.value
.map((style) => style.innerHTML)
.join("")}</style>`
);
const findAndInsertStyle = () => {
inlineStyles.value = Array.from(
document.getElementsByTagName("style")
).filter((node) => {
return node.innerHTML.includes("VueCustomElementChildren");
});
};
// @ts-ignore
if (!import.meta.env.PROD) {
const observer = new MutationObserver(findAndInsertStyle);
observer.observe(document.head, {
childList: true,
subtree: true,
characterData: true,
});
onMounted(findAndInsertStyle);
onUnmounted(() => {
observer.disconnect();
});
}
</script>
<template>
<div v-html="html"></div>
</template>
<style>
/* VueCustomElementChildren */
</style>
SomeOtherComponent.vue
<script setup lang="ts">
</script>
<template>
<div class="some-other">test</div>
</template>
<style>
/* VueCustomElementChildren */
.some-other {
color: red;
}
</style>
App.ce.vue
<script setup lang="ts">
import CustomElements from "@/components/CustomElements.vue";
import SomeOtherComponent from "@/components/SomeOtherComponent.vue";
</script>
<template>
<CustomElements css-href="/style/stylesheet.css" />
<SomeOtherComponent />
</template>
<style>
</style>
I've found a blog by @ElMassimo describing a workaround which works for me. To summarize:
- Name all SFC file names
*.ce.vue
- In your
main.ts
use defineCustomElement. - Define your custom elements.
- Use tag names instead of importing components. E.G
<HelloWorld msg="hi"/>
becomes\<hello-world msg="hi" />
.
Don't know the limitations yet e.G vuex etc.
As a workaround I've created component, which tracks Mutation in
document.head
and inline style tags (must includes comment/* VueCustomElementChildren */
) into shadow dom in dev mode, for production it inlines link for stylesheets:
@dezmound, wont that make other Vue custom elements' styles get injected into the first one?
For example, if you have 3 custom elements in the same page:
element-a
is inserted in page adds style todocument.head
- observer from
element-a
copies the style fromelement-a
to its shadow DOM element-b
is inserted in page and adds style todocument.head
- observer from
element-a
copies the style fromelement-b
toelement-a
shadow DOM - observer from
element-b
copies the style fromelement-a
andelement-b
toelement-b
shadow DOM element-c
is inserted in page and adds style todocument.head
- observer from
element-a
copies the style fromelement-c
toelement-a
shadow DOM - observer from
element-b
copies the style fromelement-c
toelement-b
shadow DOM - observer from
element-b
copies the style fromelement-a
,element-b
andelement-c
toelement-c
shadow DOM
As so you end up with unwanted styles from other components into the components. It only works well if there is only one Vue based custom element/application in the page.
As a workaround I've created component, which tracks Mutation in
document.head
and inline style tags (must includes comment/* VueCustomElementChildren */
) into shadow dom in dev mode, for production it inlines link for stylesheets:@dezmound, wont that make other Vue custom elements' styles get injected into the first one?
For example, if you have 3 custom elements in the same page:
element-a
is inserted in page adds style todocument.head
- observer from
element-a
copies the style fromelement-a
to its shadow DOMelement-b
is inserted in page and adds style todocument.head
- observer from
element-a
copies the style fromelement-b
toelement-a
shadow DOM- observer from
element-b
copies the style fromelement-a
andelement-b
toelement-b
shadow DOMelement-c
is inserted in page and adds style todocument.head
- observer from
element-a
copies the style fromelement-c
toelement-a
shadow DOM- observer from
element-b
copies the style fromelement-c
toelement-b
shadow DOM- observer from
element-b
copies the style fromelement-a
,element-b
andelement-c
toelement-c
shadow DOMAs so you end up with unwanted styles from other components into the components. It only works well if there is only one Vue based custom element/application in the page.
@claudiomedina Thank you for the comment 🙏 That's right, but it can be solved for example with adding scope prop for CustomElements.vue
:
CustomElements.vue
<script setup lang="ts">
/***
* @see https://github.com/vuejs/vue-next/issues/4662
**/
import { computed } from "vue";
import { onMounted, onUnmounted, ref } from "@vue/runtime-core";
const props = defineProps<{
cssHref?: string;
scope?: string;
}>();
const inlineStyles = ref<HTMLElement[]>([]);
const html = computed(() =>
// @ts-ignore
import.meta.env.PROD && props.cssHref
? `<link href="${props.cssHref}" rel="stylesheet"/>`
: `<style>${inlineStyles.value
.map((style) => style.innerHTML)
.join("")}</style>`
);
const findAndInsertStyle = () => {
inlineStyles.value = Array.from(
document.getElementsByTagName("style")
).filter((node) => {
return node.innerHTML.includes(`${props.scope}: VueCustomElementChildren`);
});
};
// @ts-ignore
if (!import.meta.env.PROD) {
const observer = new MutationObserver(findAndInsertStyle);
observer.observe(document.head, {
childList: true,
subtree: true,
characterData: true,
});
onMounted(findAndInsertStyle);
onUnmounted(() => {
observer.disconnect();
});
}
</script>
<template>
<div v-html="html"></div>
</template>
<style>
/* VueCustomElementChildren */
</style>
Element-a.ce.vue
<script setup lang="ts">
import CustomElements from "@/components/CustomElements.vue";
import SomeOtherComponent from "@/components/SomeOtherComponent.vue";
</script>
<template>
<CustomElements css-href="/style/stylesheet.css" scope="Element A" />
<SomeOtherComponentForA />
</template>
<style>
</style>
SomeOtherComponentForA.vue
<script setup lang="ts">
</script>
<template>
<div class="some-other">test</div>
</template>
<style>
/* Element A: VueCustomElementChildren */
.some-other {
color: red;
}
</style>
Element-b.ce.vue
<script setup lang="ts">
import CustomElements from "@/components/CustomElements.vue";
import SomeOtherComponent from "@/components/SomeOtherComponent.vue";
</script>
<template>
<CustomElements css-href="/style/stylesheet.css" scope="Element B" />
<SomeOtherComponentForB />
</template>
<style>
</style>
SomeOtherComponentForB.vue
<script setup lang="ts">
</script>
<template>
<div class="some-other">test</div>
</template>
<style>
/* Element B: VueCustomElementChildren */
.some-other {
color: red;
}
</style>
My example's just a solution that allows to write and use components for CustomElement
without casting it into native custom elements, so it's still compactable with TS. The child components may be shared with any entry points such as Vue App or different Custom Element. It doesn't contain complex logic because it does not need to. In some other cases as in your example, this workaround may be easily improved.
Is there any solution for this in sight ? This is a big show stopper in https://github.com/vitejs/vite/issues/5731
Is there any solution for this in sight ? This is a big show stopper in vitejs/vite#5731
If you don't need to use shadow DOM, you can use your own defineCustomElement
function from this PR as described here: https://github.com/vuejs/core/issues/4314#issuecomment-1021393430. That fixes the issue.
Thank's, but I need to use shadow DOM. I guess another workaround would be to use global styles from the top component - instead of scoped styles in each sub component ?
@lroal
I use global styles with importing stylesheets in root Component (root.ce.vue) to avoid this problem.
<style>
// v-bind things... like
--configurable-size: v-bind("~~~")
</style>
<style src="./styles/sheet1"></style> // maybe, use css variables declared in root component style
<style src="./styles/sheet2"></style>
<style src="./styles/sheet3"></style>
<style src="./styles/sheet4"></style>
import { createApp, defineCustomElement } from 'vue'
import App from './App.vue'
const styles = ['button { background: red; }']
customElements.define('yuty-lens', defineCustomElement({ ...App, styles }))
const app = createApp(App)
app.mount('#app')
I added inline CSS for shadow root but styles aren't loading at all, Does anyone has with the same issue?
Hi, is there a solution coming for this?
I currently use a workaround where I let vue-cli's webpack simply generate external link-tags inside a template tag. In the mounted hook I clone those tags into the shadow root.
This solution also works with @-paths for static assets in the css/scss.
I've added a minimal demo here, maybe it's helpful for someone:
https://gitlab.com/cgz/vue-web-comp-css-injection-demo.git
But I would really prefer a out of the box solution. Hopefully it will be fixed in one of the next versions.
There's another workaround:
- rename all components to
*.ce.vue
or enable{ customElement: true }
in your config so all components styles are exported like described - in your main.ts import all components you are using in your CE and concatenate their styles with the main components styles e.g.:
import HelloWorld from './components/HelloWorld.vue';
import App from './App.vue';
App.styles = [...App.styles, ...HelloWorld.styles];
customElements.define('my-component', defineCustomElement(App));
I haven't exhaustively tested this yet but it works for me with the starter app.