Skip to content

Bundle DragDropTouch locally instead of loading from external CDN#186

Merged
ProLoser merged 2 commits intomainfrom
copilot/use-local-libraries
Mar 14, 2026
Merged

Bundle DragDropTouch locally instead of loading from external CDN#186
ProLoser merged 2 commits intomainfrom
copilot/use-local-libraries

Conversation

Copy link
Contributor

Copilot AI commented Mar 14, 2026

index.html was fetching DragDropTouch.js at runtime from bernardo-castilho.github.io, creating a hard dependency on an external host for core touch drag-and-drop functionality.

Changes

  • package.json — add drag-drop-touch@1.3.1 as a dependency
  • src/index.tsx — side-effect import replaces the CDN script tag:
    import 'drag-drop-touch';
  • index.html — remove the external <script src="https://bernardo-castilho.github.io/..."> tag

The polyfill is now bundled by Vite and served locally.

Original prompt

let's use local copies of libraries instead of hosted ones


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: ProLoser <67395+ProLoser@users.noreply.github.com>
Copilot AI changed the title [WIP] Update to use local copies of libraries Bundle DragDropTouch locally instead of loading from external CDN Mar 14, 2026
Copilot AI requested a review from ProLoser March 14, 2026 19:46
@ProLoser ProLoser marked this pull request as ready for review March 14, 2026 20:12
@github-actions
Copy link

github-actions bot commented Mar 14, 2026

PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-03-14 21:35 UTC

@ProLoser ProLoser merged commit e453fab into main Mar 14, 2026
2 checks passed
@ProLoser ProLoser deleted the copilot/use-local-libraries branch March 14, 2026 21:35
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