Skip to content

fix: inspector url#4707

Merged
zyyv merged 2 commits intounocss:mainfrom
GauBen:fix/inspector-url
Jun 7, 2025
Merged

fix: inspector url#4707
zyyv merged 2 commits intounocss:mainfrom
GauBen:fix/inspector-url

Conversation

@GauBen
Copy link
Contributor

@GauBen GauBen commented Jun 6, 2025

Hi!

It seems like #4536 was not correct fix for this issue.

# Before this PR

## With base=/

  ➜  Local:   http://localhost:5173/
  ➜  UnoCSS Inspector: http:/localhost:5173/__unocss/
                            ^ notice the broken protocol

## With base=/app (through sveltekit)

  ➜  Local:   http://localhost:5173/app
  ➜  UnoCSS Inspector: http://localhost:5173/app/__unocss/
                       ^ incorrect url, the inspector will be at /__unocss/

# After this PR

## With base=/

  ➜  Local:   http://localhost:5173/
  ➜  UnoCSS Inspector: http://localhost:5173/__unocss/

## With base=/app

  ➜  Local:   http://localhost:5173/app
  ➜  UnoCSS Inspector: http://localhost:5173/__unocss/

I think my testing was careful enough, but ping me if I missed anything

@GauBen GauBen requested review from antfu and zyyv as code owners June 6, 2025 12:43
@netlify
Copy link

netlify bot commented Jun 6, 2025

Deploy Preview for unocss ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 06b357b
🔍 Latest deploy log https://app.netlify.com/projects/unocss/deploys/6842e30a4dfd2e000861cdc4
😎 Deploy Preview https://deploy-preview-4707--unocss.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Jun 6, 2025

Open in StackBlitz

commit: 06b357b

@zyyv zyyv added this pull request to the merge queue Jun 7, 2025
Merged via the queue into unocss:main with commit 0d396af Jun 7, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants