Page MenuHomePhabricator

Blueprint is doing all the work of building the regular sidebar, then ignoring it.
Closed, DeclinedPublic

Description

In getLeftNav() - Why reinvent the wheel? Replace all this code with a call to skin->addToSidebar($bar, 'blueprint-left-nav'). And you should be doing this in response to buildSidebar(), not in response to execute().

(from https://gerrit.wikimedia.org/r/#/c/205217/ )

Event Timeline

Prtksxna raised the priority of this task from to Medium.
Prtksxna updated the task description. (Show Details)
Prtksxna added a project: Blueprint.
Prtksxna added subscribers: Prtksxna, Spage.

@Prtksxna What's the status of this? In order to moving this forward, I'd need more context of the 'MediaWiki' way of doing it.

@Prtksxna What's the status of this? In order to moving this forward, I'd need more context of the 'MediaWiki' way of doing it.

I think @Spage was referring to https://www.mediawiki.org/wiki/Manual:Hooks/SkinBuildSidebar and https://doc.wikimedia.org/mediawiki-core/master/php/classSkin.html#a9c48cc7b179fbfef7d2c6a381f00954c. I am not that familiar with the PHP API, @Spage might be able to help.

The skin is no longer maintained