Address eslint errors

[JoshuaSBrown]

PR Description

Tasks

• - A description of the PR has been provided, and a diagram included if it is a new feature.
• - Formatter has been run
• - CHANGELOG comment has been added
• - Labels have been assigned to the pr
• - A reviwer has been added
• - A user has been assigned to work on the pr
• - If new feature a unit test has been added

Summary by Sourcery

Chores:

• Address ESLint errors in the jquery.js file.

[sourcery-ai]

Reviewer's Guide by Sourcery

This PR addresses eslint errors in the jquery.js file by removing or commenting out eslint directives.

Class diagram showing affected jQuery components

classDiagram
    class Sizzle {
      +setDocument(node)
      +contains(context, elem)
      +attr(elem, name)
    }
    class jQuery {
      +extend()
    }
    class Expr {
      +selectors
    }
    note for Sizzle "Modified document comparison logic
Removed eslint directives"
    note for jQuery "Modified value handling
Removed eslint directives"
    note for Expr "Modified selector handling
Removed eslint directives"

Loading

File-Level Changes

Change	Details	Files
Addressed eslint errors related to strict comparison and unused expressions.	Removed or commented out eslint-disable directives for eqeqeq and no-unused-expressions rules. Ensured compliance with the eqeqeq rule by using strict equality (===) or adding necessary comments to justify loose equality (==). Removed or commented out eslint-disable directives for no-unused-expressions rule and ensured that expressions are used correctly or removed if unnecessary.	web/static/jquery/jquery.js
Addressed eslint errors related to maximum line length (max-len).	Removed or commented out eslint-disable directives for max-len rule. Refactored code to comply with the maximum line length limit or added necessary comments to justify exceeding the limit.	web/static/jquery/jquery.js
Addressed eslint errors related to loop functions (no-loop-func).	Removed or commented out eslint-disable directives for no-loop-func rule. Refactored code to avoid creating functions within loops or added necessary comments to justify their usage.	web/static/jquery/jquery.js
Addressed eslint errors related to conditional assignment (no-cond-assign).	Removed or commented out eslint-disable directives for no-cond-assign rule. Refactored code to avoid conditional assignments or added necessary comments to justify their usage.	web/static/jquery/jquery.js

---

Tips and commands

Interacting with Sourcery

• Trigger a new review: Comment @sourcery-ai review on the pull request.
• Continue discussions: Reply directly to Sourcery's review comments.
• Generate a GitHub issue from a review comment: Ask Sourcery to create an
  issue from a review comment by replying to it.
• Generate a pull request title: Write @sourcery-ai anywhere in the pull
  request title to generate a title at any time.
• Generate a pull request summary: Write @sourcery-ai summary anywhere in
  the pull request body to generate a PR summary at any time. You can also use
  this command to specify where the summary should be inserted.

Customizing Your Experience

Access your dashboard to:

• Enable or disable review features such as the Sourcery-generated pull request
  summary, the reviewer's guide, and others.
• Change the review language.
• Add, remove or edit custom review instructions.
• Adjust other review settings.

Getting Help

• Contact our support team for questions or feedback.
• Visit our documentation for detailed guides and information.
• Keep in touch with the Sourcery team by following us on X/Twitter, LinkedIn or GitHub.

[sourcery-ai]

Hey @JoshuaSBrown - I've reviewed your changes - here's some feedback:

Overall Comments:

• Please do not remove the eslint-disable comments without addressing the underlying issues. These exceptions were intentionally added to handle browser compatibility edge cases, particularly for IE and Edge. If you want to address the eslint warnings, you'll need to either properly document why each exception is necessary, or fix the code in a way that maintains cross-browser compatibility with thorough testing.

Here's what I looked at during the review

• 🟢 General issues: all looks good
• 🟢 Security: all looks good
• 🟢 Testing: all looks good
• 🟢 Complexity: all looks good
• 🟢 Documentation: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}