Hello Jack Phoenix, Catrope, Sbisson, jenkins-bot,
I'd like you to do a code review. Please visit
https://gerrit.wikimedia.org/r/381818
to review the following change.
Change subject: Revert "RCFilters: Don't apply/clear highlights 66 times"
......................................................................
Revert "RCFilters: Don't apply/clear highlights 66 times"
This reverts commit 9f035c6a426c145794f141e233d584793e8bebf2.
Change-Id: I71330b014809331030d291b54df1799631db3994
---
M resources/src/mediawiki.rcfilters/dm/mw.rcfilters.dm.FiltersViewModel.js
1 file changed, 2 insertions(+), 4 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core
refs/changes/18/381818/1
diff --git
a/resources/src/mediawiki.rcfilters/dm/mw.rcfilters.dm.FiltersViewModel.js
b/resources/src/mediawiki.rcfilters/dm/mw.rcfilters.dm.FiltersViewModel.js
index fdea5ef..3b882a6 100644
--- a/resources/src/mediawiki.rcfilters/dm/mw.rcfilters.dm.FiltersViewModel.js
+++ b/resources/src/mediawiki.rcfilters/dm/mw.rcfilters.dm.FiltersViewModel.js
@@ -1026,14 +1026,12 @@
enable = enable === undefined ? !this.highlightEnabled : enable;
if ( this.highlightEnabled !== enable ) {
- // HACK make sure highlights are disabled globally
while we toggle on the items,
- // otherwise we'll call clearHighlight() and
applyHighlight() many many times
- this.highlightEnabled = false;
+ this.highlightEnabled = enable;
+
this.getItems().forEach( function ( filterItem ) {
filterItem.toggleHighlight(
this.highlightEnabled );
}.bind( this ) );
- this.highlightEnabled = enable;
this.emit( 'highlightChange', this.highlightEnabled );
}
};
--
To view, visit https://gerrit.wikimedia.org/r/381818
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I71330b014809331030d291b54df1799631db3994
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: wmf/1.31.0-wmf.1
Gerrit-Owner: Niharika29 <[email protected]>
Gerrit-Reviewer: Catrope <[email protected]>
Gerrit-Reviewer: Jack Phoenix <[email protected]>
Gerrit-Reviewer: Sbisson <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
