Skip to content

Fix local resource URLs with hashes#8589

Open
maxrross wants to merge 1 commit into
4ian:masterfrom
maxrross:codex/fix-file-url-hash-preview
Open

Fix local resource URLs with hashes#8589
maxrross wants to merge 1 commit into
4ian:masterfrom
maxrross:codex/fix-file-url-hash-preview

Conversation

@maxrross
Copy link
Copy Markdown

Fixes #4015.

Summary

  • Escape local Electron file URLs so # and ? in resource paths are treated as path characters instead of URL fragments/query markers.
  • Decode escaped file URLs when turning resource URLs back into local paths.
  • Add regression coverage for hashed resource paths and decoded local paths.

Tests

  • npm --prefix newIDE/app test -- --runTestsByPath src/ResourcesLoader/index.spec.js src/ResourcesList/ResourceUtils.spec.js --watchAll=false

@maxrross maxrross requested a review from 4ian as a code owner May 11, 2026 21:59
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.

[$20.00 bounty] In a filepath a hashtag broke the preview

1 participant