Skip to content

Commit

Permalink
Remove jquery entirely
Browse files Browse the repository at this point in the history
  • Loading branch information
mjgiarlo committed Oct 11, 2023
1 parent 21da018 commit e385aaf
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 6 deletions.
1 change: 0 additions & 1 deletion app/javascript/application.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
// Entry point for the build script in your package.json

import 'jquery' // Blacklight 7 needs jQuery. Remove when we upgrade to Blacklight 8?
import bootstrap from 'bootstrap/dist/js/bootstrap'
import Blacklight from 'blacklight-frontend/app/assets/javascripts/blacklight/blacklight'

Expand Down
4 changes: 0 additions & 4 deletions app/javascript/jquery-shim.js

This file was deleted.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
},
"license": "Apache-2.0",
"scripts": {
"build": "esbuild app/javascript/*.* --bundle --inject:app/javascript/jquery-shim.js --define:global=window --outdir=app/assets/builds",
"build": "esbuild app/javascript/*.* --bundle --define:global=window --outdir=app/assets/builds",
"build:css": "sass ./app/assets/stylesheets/application.scss ./app/assets/builds/application.css --no-source-map --load-path=node_modules",
"lint": "standard app/javascript/**/*.js",
"lint_fix": "standard --fix app/javascript/**/*.js"
Expand Down

0 comments on commit e385aaf

Please sign in to comment.