harlan-zw / nuxt-seo-ui

Deprecated - Nuxt components with Schema.org and SEO integrations
5 stars 1 forks source link

fix: breadcrumbs - ui not included #2

Closed dargmuesli closed 1 year ago

dargmuesli commented 1 year ago

Describe the bug

Making use of useBreadcrumbsUi does not lead to the defined tailwind classes being included into the existing (@nuxtjs/tailwindcss) stylesheet as expected.

image

Here, only the classes that I use somewhere else in my app are applied, not all, that's why the separators are not positioned correctly.


I'd love to know how to implement this correctly, I really want to make use of the same styling in @dargmuesli/nuxt-cookie-control :heart_eyes:

Reproduction

No response

System / Nuxt Info

No response

harlan-zw commented 1 year ago

Hm, the classes should be picked up as the tailwind config is merged within the hook. I just pushed a fix up if you have config.content as a string of arrays though.

 nuxt.hooks.hook('tailwindcss:config', (config) => {
      if (Array.isArray(config.content)) {
        // add our components for scanning
        config.content.push(resolve('./runtime/components/*.vue'))
        config.content.push(resolve('./runtime/composables/useBreadcrumbsUi.ts'))
      }
      else if (config.content?.files && Array.isArray(config.content?.files)) {
        // add our components for scanning
        config.content.files.push(resolve('./runtime/components/*.vue'))
        config.content.files.push(resolve('./runtime/composables/useBreadcrumbsUi.ts'))
      }
    })

Maybe you can see if that fixed it for you :thinking:

dargmuesli commented 1 year ago

LGTM! Thank you :)