MediaWiki talk:Gadget-contribsrange.js

Archives

This page has archives. Sections older than 180 days may be automatically archived by when more than 3 sections are present.

fix getElementsByClassName

{{Editprotected}} see [1], fix 'toggle all'. ref. --YFdyh000 (talk) 07:07, 9 January 2014 (UTC)[reply]

That's not the right fix. It will break on older browsers. We should use jquery. —TheDJ (talk • contribs) 18:13, 14 January 2014 (UTC)[reply]

done using jquery —TheDJ (talk • contribs) 18:18, 14 January 2014 (UTC)[reply]

oh, thanks for remind, jquery is a good idea. merely it seems only break for IE6-IE8.--YFdyh000 (talk) 05:38, 16 January 2014 (UTC)[reply]

Still necessary

Is this entire gadget still necessary now that CIDR ranges are supported natively? I note that wildcards are permitted by this gadget. It might be a good idea to remove the functionality in this gadget which is presently unnecessary. --Izno (talk) 19:52, 22 November 2017 (UTC)[reply]

@Izno: This gadget groups edits by IP, while the native implementation doesn’t. For me the grouped version is more usable. This is, of course, subjective, but I think it’s enough to justify keeping this feature in the gadget. —Tacsipacsi (talk) 12:15, 12 April 2020 (UTC)[reply]

Interface-protected edit request on 25 December 2019

This edit request has been answered. Set the |answered= or |ans= parameter to no to reactivate your request.

In the comment on line 21, please replace "These parameter soverride" with "These parameters override" (i.e. move the "s" to the other side of the space). Thanks, --DannyS712 (talk) 01:36, 25 December 2019 (UTC)[reply]

Done Galobtter (pingó mió) 08:30, 25 December 2019 (UTC)[reply]

Spinner

It is requested that an edit be made to the interface page at MediaWiki:Gadget-contribsrange.js. (edit · history · last · links)

This template must be followed by a complete and specific description of the request, that is, specify what text should be removed and a verbatim copy of the text that should replace it. "Please change X" is not acceptable and will be rejected; the request must be of the form "please change X to Y".

The edit may be made by any interface administrator. Remember to change the |answered=no parameter to "yes" when the request has been accepted, rejected or on hold awaiting user input. This is so that inactive or completed requests don't needlessly fill up the edit requests category. You may also wish to use the {{EIp}} template in the response. When the request has been completed or denied, please add the |answered=yes parameter to deactivate the template.

Jon (WMF) changed the code so that it only tries to use the jQuery spinner if that’s already loaded. While this fix avoids null dereference, I don’t think it’s a good solution: if we want to show a spinner, we want to show it always, not just when it happens to have been loaded. Therefore I suggest changing

  if (mw.loader.getState('jquery.spinner') === 'ready' ) {
    $(spin).injectSpinner('prefixcontribs-spin');
  }

to

  mw.loader.using('jquery.spinner').then(function () {
    $(spin).injectSpinner('prefixcontribs-spin');
  });

which makes sure that the spinner always appears sooner or later. Thanks in advance, —Tacsipacsi (talk) 11:04, 13 September 2020 (UTC)[reply]

If

mw.loader.getState('jquery.spinner') === 'ready'

is not truthy than the next line will always throw a JS error. Adding a dependency to

mw.loader.using('jquery.spinner')

is definitely the writer solution, however you probably want to load it earlier in the script. If you are trying to display a spinner, it doesn't make sense to asynchronously show it - as that seems to defeat the purpose to me :) The spinner may display after the action the spinner is supposed to be displayed for as shown! Wrapping the whole code is one option, but ideally you'd wrap whatever begins the script e.g. the click of the UI button. Also consider not using jquery spinner for something so simple.

$('<img>').attr('src', urlOfCommonsSpinner ).appendTo(spin)

should work just as well without the overhead.Jon (WMF) (talk) 18:19, 13 September 2020 (UTC)[reply]