Sfoglia il codice sorgente

Merge pull request #18195 from overleaf/revert-18120-td-bs5-load-js

Revert "Load correct JS for the active Bootstrap version"

GitOrigin-RevId: 7f6e846b5461cfbacec874ed55bba577e414f3a6
Tim Down 2 anni fa
parent
commit
a452e1e8cd

+ 0 - 3
services/web/app/views/_mixins/bootstrap_js.pug

@@ -1,3 +0,0 @@
-mixin bootstrap-js(bootstrapVersion)
-	each file in (entrypointScripts(bootstrapVersion === 5 ? 'bootstrap-5' : 'bootstrap-3'))
-		script(type="text/javascript", nonce=scriptNonce, src=file)

+ 2 - 2
services/web/app/views/layout-base.pug

@@ -4,7 +4,7 @@ html(
 	class=(fixedSizeDocument ? 'fixed-size-document' : undefined)
 )
 	- metadata = metadata || {}
-	- let bootstrap5EnabledPage = false
+	- bootstrap5EnabledPage = false
 
 	block entrypointVar
 
@@ -13,7 +13,7 @@ html(
 	head
 		include ./_metadata.pug
 			
-		- const bootstrapVersion = bootstrap5EnabledPage && splitTestVariants['bootstrap-5'] === 'enabled' ? 5 : 3
+		- var bootstrapVersion = bootstrap5EnabledPage && splitTestVariants['bootstrap-5'] === 'enabled' ? 5 : 3
 
 		//- Stylesheet
 		link(rel='stylesheet', href=buildCssPath(getCssThemeModifier(userSettings, brandVariation), bootstrapVersion), id="main-stylesheet")

+ 0 - 4
services/web/app/views/layout-marketing.pug

@@ -1,7 +1,6 @@
 extends ./layout-base
 
 include ./_mixins/formMessages
-include ./_mixins/bootstrap_js
 
 block entrypointVar
 	- entrypoint = 'marketing'
@@ -22,6 +21,3 @@ block body
 		include _cookie_banner
 
 	!= moduleIncludes("contactModal-marketing", locals)
-
-block prepend foot-scripts
-	+bootstrap-js(bootstrapVersion)

+ 0 - 4
services/web/app/views/layout-website-redesign.pug

@@ -1,7 +1,6 @@
 extends ./layout-base
 
 include ./_mixins/formMessages
-include ./_mixins/bootstrap_js
 
 block entrypointVar
 	- entrypoint = 'marketing'
@@ -22,6 +21,3 @@ block body
 		include _cookie_banner
 
 	!= moduleIncludes("contactModal-marketing", locals)
-
-block prepend foot-scripts
-	+bootstrap-js(bootstrapVersion)

+ 0 - 4
services/web/app/views/subscriptions/interstitial-payment.pug

@@ -1,7 +1,6 @@
 extends ../layout
 
 include ./plans/_mixins
-include ../_mixins/bootstrap_js
 
 block vars
 	- entrypoint = 'pages/user/subscription/plans-v2/plans-v2-main'
@@ -66,6 +65,3 @@ block content
 					| #{translate("continue_with_free_plan")}			
 
 	!= moduleIncludes("contactModalGeneral-marketing", locals)
-
-block prepend foot-scripts
-	+bootstrap-js(bootstrapVersion)

+ 0 - 5
services/web/frontend/js/bootstrap-3.ts

@@ -1,5 +0,0 @@
-import 'jquery'
-import 'bootstrap'
-
-$('[data-ol-lang-selector-tooltip]').tooltip({ trigger: 'hover' })
-$('[data-toggle="tooltip"]').tooltip()

+ 0 - 1
services/web/frontend/js/bootstrap-5.ts

@@ -1 +0,0 @@
-import 'bootstrap-5'

+ 5 - 0
services/web/frontend/js/marketing.js

@@ -1,5 +1,7 @@
 import './utils/webpack-public-path'
 import './infrastructure/error-reporter'
+import 'jquery'
+import 'bootstrap'
 import './features/form-helpers/hydrate-form'
 import './features/form-helpers/password-visibility'
 import './features/link-helpers/slow-link'
@@ -11,3 +13,6 @@ import './features/multi-submit'
 import './features/cookie-banner'
 import './features/autoplay-video'
 import './features/mathjax'
+
+$('[data-ol-lang-selector-tooltip]').tooltip({ trigger: 'hover' })
+$('[data-toggle="tooltip"]').tooltip()

+ 2 - 0
services/web/frontend/js/pages/project-list.jsx

@@ -1,3 +1,5 @@
+import 'jquery'
+import 'bootstrap'
 import './../utils/meta'
 import './../utils/webpack-public-path'
 import './../infrastructure/error-reporter'

+ 0 - 2
services/web/webpack.config.js

@@ -17,8 +17,6 @@ invalidateBabelCacheIfNeeded()
 // Generate a hash of entry points, including modules
 const entryPoints = {
   tracing: './frontend/js/tracing.js',
-  'bootstrap-3': './frontend/js/bootstrap-3.ts',
-  'bootstrap-5': './frontend/js/bootstrap-5.ts',
   devToolbar: './frontend/js/dev-toolbar.js',
   main: './frontend/js/main.js',
   ide: './frontend/js/ide.js',