RCFilters: Followup I78e23f85c3: Don't disable RCFilters system when fetching results
authorMoriel Schottlender <moriel@gmail.com>
Wed, 26 Jul 2017 03:34:47 +0000 (20:34 -0700)
committerMoriel Schottlender <moriel@gmail.com>
Wed, 26 Jul 2017 03:38:59 +0000 (20:38 -0700)
We disable and hide it only on first page load while we create the
system, but should not hide and disable the actual filter area
system.

Bug: T165286
Bug: T171671
Change-Id: Ifc9d3c02c0c118d48634b8df846e14bfa6ea294a

resources/src/mediawiki.rcfilters/mw.rcfilters.init.js
resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.ChangesListWrapperWidget.js
resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.FormWrapperWidget.js

index 4263c51..e758f26 100644 (file)
@@ -36,6 +36,7 @@
 
                        $( '.rcfilters-container' ).append( filtersWidget.$element );
                        $( 'body' ).append( $overlay );
+                       $( '.rcfilters-head' ).addClass( 'mw-rcfilters-ui-ready' );
 
                        $( 'a.mw-helplink' ).attr(
                                'href',
index 37dda4d..c2533df 100644 (file)
@@ -76,7 +76,6 @@
         * Respond to changes list model invalidate
         */
        mw.rcfilters.ui.ChangesListWrapperWidget.prototype.onModelInvalidate = function () {
-               $( '.rcfilters-head' ).removeClass( 'mw-rcfilters-ui-ready' );
                $( '.rcfilters-spinner' ).removeClass( 'mw-rcfilters-ui-ready' );
                this.$element.removeClass( 'mw-rcfilters-ui-ready' );
        };
                        }
                }
 
-               $( '.rcfilters-head' ).addClass( 'mw-rcfilters-ui-ready' );
                $( '.rcfilters-spinner' ).addClass( 'mw-rcfilters-ui-ready' );
                this.$element.addClass( 'mw-rcfilters-ui-ready' );
        };
index 04ccaf6..ee8e0bc 100644 (file)
@@ -87,6 +87,7 @@
         */
        mw.rcfilters.ui.FormWrapperWidget.prototype.onChangesModelInvalidate = function () {
                this.$submitButton.prop( 'disabled', true );
+               this.$element.removeClass( 'mw-rcfilters-ui-ready' );
        };
 
        /**
         */
        mw.rcfilters.ui.FormWrapperWidget.prototype.onChangesModelUpdate = function ( $changesList, $fieldset, isInitialDOM ) {
                this.$submitButton.prop( 'disabled', false );
+               this.$element.removeClass( 'mw-rcfilters-ui-ready' );
 
                // Replace the entire fieldset
                this.$element.empty().append( $fieldset.contents() );