Page MenuHomePhabricator

Close button is missing from the meta-data dialogs
Closed, ResolvedPublic

Description

Event Timeline

Change #1215205 had a related patch set uploaded (by Daphne Smit; author: Daphne Smit):

[mediawiki/extensions/WikiLambda@master] CdxDialog: use-close-button prop needs to be set to true

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

DSmit-WMF changed the task status from Open to In Progress.Dec 4 2025, 4:32 PM
DSmit-WMF claimed this task.

I think this may have been caused by Codex 2.3.3 which is new this release (wmf.5) — looking at the changelog shows some entries around the close button control. Ah well.

Change #1215205 merged by jenkins-bot:

[mediawiki/extensions/WikiLambda@master] CdxDialog: use-close-button prop needs to be set to true

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

Change #1215224 had a related patch set uploaded (by Jforrester; author: Daphne Smit):

[mediawiki/extensions/WikiLambda@wmf/1.46.0-wmf.5] CdxDialog: use-close-button prop needs to be set to true

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

Sorry about this -- the reason this broke is because we changed the way we hide the close button from using v-if to using CSS. The previous way (v-if) didn't affect you because you override the header with a custom one so we don't render our own close button regardless. The new way (CSS) did affect you, because your close button in your custom header uses the same CSS class as our built-in one.

Change #1215224 merged by jenkins-bot:

[mediawiki/extensions/WikiLambda@wmf/1.46.0-wmf.5] CdxDialog: use-close-button prop needs to be set to true

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