jthingelstad / foreground

A MediaWiki skin that focuses on putting your content in the foreground.
http://foreground.wikiproject.net/
Other
137 stars 60 forks source link

Broken with Mediawiki 1.37.1 [solved] #395

Open fw25 opened 2 years ago

fw25 commented 2 years ago

Setup

Issue

Detailed description of the issue goes here. There is an jquery error in a javascript file it is about url.indexOf. It cripples the internal links. Screen Shot 2022-03-10 at 16 20 05

fw25 commented 2 years ago

There is a simple workaround. Replace load with on(“load”

assets/scripts/foundation/foundation.topbar.js replace in line 248 }, 50)).trigger('resize.fndtn.topbar').load(function () { with }, 50)).trigger('resize.fndtn.topbar’).on(“load”,function () {

assets/scripts/foundation/foundation.js replace line 345 S(window).load(function () { with S(window).on("load",function () {

racribeiro commented 2 years ago

Yes, this works.

Please consider pushing this change to the repository.

Hutchy68 commented 2 years ago

If someone opens a pr against develop branch I’ll merge.

rebastion commented 2 years ago

@fw25 it does give a "Uncaught SyntaxError: '' string literal contains an unescaped line break" error in the debugger though, did I copy paste something wrong?

rebastion commented 2 years ago

Works if done like this https://github.com/jthingelstad/foreground/commit/82bbd1539eb628882eeaff8b8f16bcc2e50d098f