Skip to content

fix(css): include .css?url in assets field of manifest#17623

Merged
patak-cat merged 2 commits into
vitejs:mainfrom
sapphi-red:fix/css-url-manifest-assets
Jul 15, 2024
Merged

fix(css): include .css?url in assets field of manifest#17623
patak-cat merged 2 commits into
vitejs:mainfrom
sapphi-red:fix/css-url-manifest-assets

Conversation

@sapphi-red
Copy link
Copy Markdown
Member

Description

fixes #17484

The manifest in the repro changes from

  "_stuff.js": {
    "file": "chunks/stuff.js",
    "name": "stuff",
    "assets": [
      "assets/text.txt"
    ]
  }

to

  "_stuff.js": {
    "file": "chunks/stuff.js",
    "name": "stuff",
    "assets": [
      "assets/text.txt",
      "assets/styles.css"
    ]
  },

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat: css p3-minor-bug An edge case that only affects very specific usage (priority)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CSS files are incorrectly excluded from assets array in manifest.json

2 participants