Remove mentions of jquery.mw-jump in code comments
authorTimo Tijhof <krinklemail@gmail.com>
Thu, 24 May 2018 12:35:50 +0000 (13:35 +0100)
committerTimo Tijhof <krinklemail@gmail.com>
Thu, 24 May 2018 12:35:50 +0000 (13:35 +0100)
The idea of listening for an event and doing something, with on('focus'),
is rather trivial and not worth attributing to another module, in particular
when that module is being phased out :)

Bug: T195256
Change-Id: Ic6a65254cbcc5e9503fece1afb7861a706713d77

resources/src/mediawiki.page.gallery.js
resources/src/mediawiki.special.preferences.ooui/tabs.js
resources/src/mediawiki.special.preferences/tabs.legacy.js

index 9448ab8..7927f20 100644 (file)
                } else {
                        // Note use of just `a`, not `a.image`, since we also want this to trigger if a link
                        // within the caption text receives focus.
-                       // This is based on code from the 'jquery.mw-jump' module.
                        $content.find( 'ul.mw-gallery-packed-hover li.gallerybox' ).on( 'focus blur', 'a', function ( e ) {
                                // Confusingly jQuery leaves e.type as focusout for delegated blur events
                                var gettingFocus = e.type !== 'blur' && e.type !== 'focusout';
index 40c9df9..795a2b7 100644 (file)
@@ -8,8 +8,8 @@
                $preferences = $( '#preferences' );
 
                // Make sure the accessibility tip is selectable so that screen reader users take notice,
-               // but hide it per default to reduce interface clutter. Also make sure it becomes visible
-               // when selected. Similar to jquery.mw-jump
+               // but hide it by default to reduce visual clutter.
+               // Make sure it becomes visible when focused.
                $( '<div>' ).addClass( 'mw-navigation-hint' )
                        .text( mw.msg( 'prefs-tabs-navigation-hint' ) )
                        .attr( 'tabIndex', 0 )
index 0d97d68..598b8f8 100644 (file)
@@ -27,8 +27,8 @@
                $fieldsets.children( 'legend' ).addClass( 'mainLegend' );
 
                // Make sure the accessibility tip is selectable so that screen reader users take notice,
-               // but hide it per default to reduce interface clutter. Also make sure it becomes visible
-               // when selected. Similar to jquery.mw-jump
+               // but hide it by default to reduce visual clutter.
+               // Make sure it becomes visible when focused.
                $( '<div>' ).addClass( 'mw-navigation-hint' )
                        .text( mw.msg( 'prefs-tabs-navigation-hint' ) )
                        .attr( 'tabIndex', 0 )