From 3dbc5749eaac40bb1c951de63308882acfcc1d78 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bartosz=20Dziewo=C5=84ski?= Date: Tue, 22 Jun 2021 19:01:52 +0200 Subject: [PATCH] Adjust use of 'plainlinks' class to fix button styling in some cases It was previously applied on the whole wrapper of the anon warning, including the buttons. In some cases (on mobile, and on some sites with weird customized site CSS), this causes the buttons to lose some styling. Apply it only on the label. Follow-up to 522b7932d7afc0e2c1646554cef20b1cf0936b50. Bug: T270536 Bug: T291000 Change-Id: Ia4bc99fc219a80efbf46a7cc196ea29720a34de8 --- modules/dt.ui.ReplyWidget.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/modules/dt.ui.ReplyWidget.js b/modules/dt.ui.ReplyWidget.js index 5d4237a25..6da067403 100644 --- a/modules/dt.ui.ReplyWidget.js +++ b/modules/dt.ui.ReplyWidget.js @@ -212,7 +212,7 @@ function ReplyWidget( commentController, comment, commentDetails, config ) { returnto: mw.config.get( 'wgPageName' ) }; this.anonWarning = new OO.ui.MessageWidget( { - classes: [ 'ext-discussiontools-ui-replyWidget-anonWarning plainlinks' ], + classes: [ 'ext-discussiontools-ui-replyWidget-anonWarning' ], type: 'warning', label: mw.message( 'discussiontools-replywidget-anon-warning' ) .params( [ @@ -222,6 +222,7 @@ function ReplyWidget( commentController, comment, commentDetails, config ) { .parseDom() } ); this.anonWarning.$element.append( this.$actions ); + this.anonWarning.$label.addClass( 'plainlinks' ); this.$element.append( this.anonWarning.$element, this.$footer ); this.$actionsWrapper.detach(); }