Skip to content

remove scripts adding through code - #18

Merged
FO-nTTaX merged 2 commits into
masterfrom
remove-scripts-adding
Mar 10, 2026
Merged

remove scripts adding through code#18
FO-nTTaX merged 2 commits into
masterfrom
remove-scripts-adding

Conversation

@godarkrai

Copy link
Copy Markdown
Member

We will load UseLoader.js, Core.js and CoreEnd.js through database directly with their priorities set so that they are still loaded in order they are right now.

ONLY MERGE ONCE THE SCRIPTS ARE ADDED THROUGH THE Special:RLA special page.

@godarkrai
godarkrai requested a review from FO-nTTaX March 9, 2026 15:44
Comment thread src/Hooks/MainHookHandler.php Outdated
$wikiUrl = $config->get( 'ResourceLoaderArticlesWiki' );
$scripts = [ 'UseStrict.js', 'Polyfill.js', 'Core.js' ];
$scripts = [];
$styles = [ 'Variables.css' ];

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

let's remove Variables.css too

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

removed

@FO-nTTaX
FO-nTTaX merged commit 9b61784 into master Mar 10, 2026
4 checks passed
@FO-nTTaX
FO-nTTaX deleted the remove-scripts-adding branch March 10, 2026 10:07
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.

2 participants