Page MenuHomePhabricator

Remove useScroll and scrollUtils if not used
Closed, ResolvedPublic

Description

Description

composables/useScroll and utils/scrollUtils don't seem to be referenced anywhere. If they aren't used, we should remove them

 
$> rg scrollUtils

ext.wikilambda.app/utils/scrollUtils.js
8:const scrollUtils = {
160:module.exports = scrollUtils;

ext.wikilambda.app/composables/useScroll.js
11:const scrollUtils = require( '../utils/scrollUtils.js' );
31:		return scrollUtils.scrollToElement( elementId, options );
41:		return scrollUtils.scrollToCurrentHash( options );
52:		return scrollUtils.scrollToElementWithRetry( elementId, options );
63:		return scrollUtils.scrollToCurrentHashWithRetry( options );
82:		debouncedHashScroll.value = scrollUtils.createDebouncedHashScroll( options );

&> rg useScroll

composables/useScroll.js
4: * @module ext.wikilambda.app.composables.useScroll
19:module.exports = function useScroll() {

Completion checklist

Event Timeline

Change #1306987 had a related patch set uploaded (by Zaidusyy; author: Zaidusyy):

[mediawiki/extensions/WikiLambda@master] WikiLambda: Remove unused scroll utilities

https://gerrit.wikimedia.org/r/1306987

Zaidusyy changed the task status from Open to In Progress.Jul 1 2026, 10:03 PM
Zaidusyy claimed this task.

Change #1306987 merged by jenkins-bot:

[mediawiki/extensions/WikiLambda@master] WikiLambda: Remove unused scroll utilities

https://gerrit.wikimedia.org/r/1306987

The patch merged in wmf.12 and has deployed. Should I mark this Resolved, or does
the team prefer to close it after your own deploy verification? Happy either way.

Jdforrester-WMF subscribed.

Sorry for the slow action, many of us at are Wikimania 2026.