Skip to content

Conversation

@mlinksva
Copy link
Contributor

Fixes #1139

Copilot AI review requested due to automatic review settings December 14, 2025 05:08
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes the ancient jQuery dependency (v1.11.0) and replaces all jQuery usage with vanilla JavaScript. The changes modernize the codebase by using native DOM APIs, the Fetch API, and modern JavaScript features.

Key Changes:

  • Complete removal of jQuery library files and dependencies
  • Refactoring of DOM selection using querySelector and querySelectorAll
  • Event handling migrated from jQuery's .on() to native addEventListener
  • AJAX requests replaced with modern Fetch API
  • DOM manipulation updated to use native classList and setAttribute methods

Reviewed changes

Copilot reviewed 3 out of 15 changed files in this pull request and generated no comments.

File Description
bower.json Removed jQuery dependency from package dependencies
assets/vendor/jquery/* Removed entire jQuery vendor directory including source, minified, and migration files
assets/js/app.js Refactored all jQuery code to vanilla JavaScript with proper event handling and DOM manipulation
_includes/footer.html Removed jQuery script tag from footer template

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mlinksva mlinksva merged commit ef0e0e2 into gh-pages Dec 14, 2025
10 of 11 checks passed
@mlinksva mlinksva deleted the qjquery branch December 14, 2025 05:13
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.

Update or replace very old vendored version of jquery

2 participants