RCFilters: Hide highlight containers when RCFilters is disabled
authorRoan Kattouw <roan.kattouw@gmail.com>
Fri, 29 Jun 2018 04:43:04 +0000 (21:43 -0700)
committerRoan Kattouw <roan.kattouw@gmail.com>
Fri, 29 Jun 2018 04:43:04 +0000 (21:43 -0700)
The mw-rcfilters-ui-highlights divs are given display: none; (if
highlights are disabled) or display: inline-block; (if they are enabled)
by the RCFilters CSS, but none of that CSS loads when RCFilters is
disabled, so in that case they default to display: block;. I don't know
why this doesn't cause them to break the line in Chrome and Firefox, but
I suspect that this is what causes the display issues in Edge that
people have been reporting.

This is a speculative fix because I'm not able to run Edge to verify.

Bug: T198440
Change-Id: I7b4d9b01d9b78828c273895a0093a2a7c0b4fc99

resources/src/mediawiki.special.changeslist.css

index 65860ea..1b37ec3 100644 (file)
@@ -54,3 +54,9 @@
 .mw-changeslist-legend.mw-collapsed + h4 + div > table.mw-changeslist-line:first-child {
        clear: right;
 }
+
+/* Hide RCFilters highlight containers if RCFilters is not enabled.
+   This is overridden in mw.ui.rcfilters.ChangesListWrapperWidget.less if RCFilters is enabled. */
+.mw-rcfilters-ui-highlights {
+       display: none;
+}