Use of Html::successBox doesn't seem to be styled on Special:AccountSecurity
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Reedy
	Sep 27 2025, 4:31 PM

Project Tags

Referenced Files

	F66704358: Screenshot 2025-09-27 at 17.28.23.png
	Sep 27 2025, 4:31 PM

Subscribers

Aklapper

Catrope

Reedy

Description

Added in rEOAT75c6f4eb9529: New UI for Special:AccountSecurity for T401774: Redesign UI for listing 2FA methods and adding new methods/T403683: Rename 2FA methods to friendlier names, the use of Html::successBox doesn't result in anything styled.. Should it?

It's fairly unnoticeable at the top as is.

Screenshot 2025-09-27 at 17.28.23.png (329×196 px, 14 KB)

Noticed while testing the patch for T404314: Redirect to Special:AccountSecurity once a new authentication method is added which reuses the same code, and displays similarly... not very noticeably.

<div class="cdx-message--success cdx-message cdx-message--block"><span class="cdx-message__icon"></span><div class="cdx-message__content"><strong>Authenticator app</strong> removed.</div></div>

Details

Related Changes in Gerrit:

	Subject	Author	Repo	Branch	Lines +/-
	Load CdxMessage styles as part of Special:OATHManage styles	Catrope	mediawiki/extensions/OATHAuth	master	+1 -0

Customize query in gerrit

Related Objects

Mentioned Here: rEOAT75c6f4eb9529: New UI for Special:AccountSecurity
T401774: Redesign UI for listing 2FA methods and adding new methods
T403683: Rename 2FA methods to friendlier names
T404314: Redirect to Special:AccountSecurity once a new authentication method is added

Event Timeline

Reedy created this task.Sep 27 2025, 4:31 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptSep 27 2025, 4:31 PM

Reedy updated the task description. (Show Details)Sep 27 2025, 4:32 PM

Reedy renamed this task from Use of Html::successBox doesn't seem to be styled to Use of Html::successBox doesn't seem to be styled on Special:AccountSecurity.Sep 27 2025, 4:48 PM

Reedy triaged this task as Low priority.

Change #1193007 had a related patch set uploaded (by Catrope; author: Catrope):

[mediawiki/extensions/OATHAuth@master] Load CdxMessage styles as part of Special:OATHManage styles

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

gerritbot added a project: Patch-For-Review.Oct 2 2025, 3:48 AM

It worked for me locally, but it looks like that was because I use Vector-2022, which already ships the cdx-message styles because it uses them for something else. In legacy Vector (and probably every other skin), it was broken per your screenshot.

Catrope claimed this task.Oct 2 2025, 3:48 AM

Catrope moved this task from Backlog to In Progress on the FY2025-26 WE4.6.2 Multiple Authenticators board.

Reedy closed this task as Resolved.Oct 2 2025, 9:43 AM

Change #1193007 merged by jenkins-bot:

[mediawiki/extensions/OATHAuth@master] Load CdxMessage styles as part of Special:OATHManage styles

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

ReleaseTaggerBot added a project: MW-1.45-notes (1.45.0-wmf.22; 2025-10-07).Oct 2 2025, 10:00 AM

Maintenance_bot removed a project: Patch-For-Review.Oct 2 2025, 10:33 AM

Catrope moved this task from In Progress to Done on the FY2025-26 WE4.6.2 Multiple Authenticators board.Oct 22 2025, 4:47 AM