Skip to content

Remove jQuery from the IndexedDB demo - #391

Open
oiahoon wants to merge 1 commit into
mdn:mainfrom
oiahoon:fix/indexeddb-native-dom
Open

Remove jQuery from the IndexedDB demo#391
oiahoon wants to merge 1 commit into
mdn:mainfrom
oiahoon:fix/indexeddb-native-dom

Conversation

@oiahoon

@oiahoon oiahoon commented Jul 15, 2026

Copy link
Copy Markdown

Fixes #370

Summary

  • remove the external jQuery 1.8.3 dependency from the IndexedDB demo
  • replace its DOM updates, event handlers, and form access with native browser APIs
  • preserve add/list/delete/clear behavior and HTML, image, PDF, and fallback blob previews

Validation

  • reproduced the initialization failure with the jQuery CDN request blocked
  • verified the complete demo offline: required-field handling, add/list/delete/clear, uploaded HTML/image/fallback previews, and same-origin URL image loading
  • verified uploaded HTML renders without executing embedded scripts
  • node --check indexeddb-api/main.js
  • tidy -errors -quiet indexeddb-api/index.html
  • git diff --check

@oiahoon
oiahoon requested review from a team as code owners July 15, 2026 14:31
@oiahoon
oiahoon requested a review from pepelsbey July 15, 2026 14:31
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.

indexeddb-api assumes familiarity with jQuery

2 participants