Page MenuHomePhabricator

"Responsive References" configuration should be exported by a hook
Closed, ResolvedPublic

Description

(Maybe this will be moot once the Parsoid implementation of the Cite extension moves into the Cite repo.)

Currently Parsoid's SiteConfig::responsiveReferences needs to parse Cite's configuration options ($wgCiteResponsiveReferences) in order to determine whether responsive references are enabled.

Cite should instead export this in a manner that doesn't require Parsoid to know the details of Cite's configuration options. Perhaps a Hook, perhaps a Service, perhaps a static method...

Event Timeline

ssastry triaged this task as Medium priority.Nov 30 2020, 6:06 PM
ssastry moved this task from Needs Triage to Following Quarter on the Parsoid board.

Change 837754 had a related patch set uploaded (by Subramanya Sastry; author: Subramanya Sastry):

[mediawiki/core@master] SiteConfig: Transition away from hardcoded Cite info

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

Change 837755 had a related patch set uploaded (by Subramanya Sastry; author: Subramanya Sastry):

[mediawiki/services/parsoid@master] SiteConfig: Transition away from hardcoded Cite info

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

Change 837754 merged by jenkins-bot:

[mediawiki/core@master] SiteConfig: Transition away from hardcoded Cite info

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

Change 837755 merged by jenkins-bot:

[mediawiki/services/parsoid@master] SiteConfig: Transition away from hardcoded Cite info

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

Change 892515 had a related patch set uploaded (by Isabelle Hurbain-Palatin; author: Isabelle Hurbain-Palatin):

[mediawiki/vendor@master] Bump parsoid to 0.17.0-a18

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

Change 892515 merged by jenkins-bot:

[mediawiki/vendor@master] Bump parsoid to 0.17.0-a18

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

Change 894768 had a related patch set uploaded (by Subramanya Sastry; author: Subramanya Sastry):

[mediawiki/services/parsoid@master] Get rid of Cite-specific SiteConfig method

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

Change 894769 had a related patch set uploaded (by Subramanya Sastry; author: Subramanya Sastry):

[mediawiki/core@master] SiteConfig: Get rid of of Cite-specific method

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

Change 894768 merged by jenkins-bot:

[mediawiki/services/parsoid@master] Get rid of Cite-specific SiteConfig method

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

Change 896413 had a related patch set uploaded (by C. Scott Ananian; author: C. Scott Ananian):

[mediawiki/vendor@master] Bump parsoid to 0.17.0-a20 and zest-css to 3.0.0

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

Change 896413 merged by jenkins-bot:

[mediawiki/vendor@master] Bump parsoid to 0.17.0-a20 and zest-css to 3.0.0

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

Change 894769 merged by jenkins-bot:

[mediawiki/core@master] SiteConfig: Get rid of of Cite-specific method

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

Change 901277 had a related patch set uploaded (by Subramanya Sastry; author: Subramanya Sastry):

[mediawiki/core@REL1_40] SiteConfig: Get rid of of Cite-specific method

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

Change 901277 merged by jenkins-bot:

[mediawiki/core@REL1_40] SiteConfig: Get rid of of Cite-specific method

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