X-Git-Url: https://git.heureux-cyclage.org/?p=lhc%2Fweb%2Fwiklou.git;a=blobdiff_plain;f=resources%2Fsrc%2Fmediawiki.action%2Fmediawiki.action.edit.js;h=966b5bcb90684535b9c45ab43711218c2214d1a6;hp=5b810174bf7d7020373d902754f49fa2617f75ec;hb=fe94275c8fcfc248a5eae857dde7c5772d993ab5;hpb=43a548a5c3436b5d07091b504bbf5eb80d1ed215 diff --git a/resources/src/mediawiki.action/mediawiki.action.edit.js b/resources/src/mediawiki.action/mediawiki.action.edit.js index 5b810174bf..966b5bcb90 100644 --- a/resources/src/mediawiki.action/mediawiki.action.edit.js +++ b/resources/src/mediawiki.action/mediawiki.action.edit.js @@ -1,7 +1,7 @@ /*! * Scripts for action=edit at domready */ -( function ( mw, $ ) { +( function () { 'use strict'; /** @@ -17,24 +17,20 @@ */ $( function () { - var editBox, scrollTop, $editForm; + var editBox, scrollTop, $editForm, + summaryCodePointLimit = mw.config.get( 'wgCommentCodePointLimit' ), + summaryByteLimit = mw.config.get( 'wgCommentByteLimit' ), + wpSummary = OO.ui.infuse( $( '#wpSummaryWidget' ) ); - if ( $( '#editform' ).hasClass( 'mw-editform-ooui' ) ) { - mw.loader.using( 'oojs-ui-core' ).then( function () { - var wpSummary = OO.ui.infuse( $( '#wpSummaryWidget' ) ); - - // Restore appropriate modifier keys for the accesskey in the 'title' attribute - // TODO: This should be an OOjs UI feature, or somehow happen automatically after infusing. - wpSummary.$input.updateTooltipAccessKeys(); - - // Show a byte-counter to users with how many bytes are left for their edit summary. - // TODO: This looks a bit weird, as there is no unit in the UI, just numbers; showing - // 'bytes' confused users in testing, and showing 'chars' would be a lie. See T42035. - mw.widgets.visibleByteLimit( wpSummary, 255 ); - } ); - } else { - // Make sure edit summary does not exceed byte limit - $( '#wpSummary' ).byteLimit( 255 ); + // Show a byte-counter to users with how many bytes are left for their edit summary. + // TODO: This looks a bit weird, as there is no unit in the UI, just numbers; showing + // 'bytes' confused users in testing, and showing 'chars' would be a lie. See T42035. + // (Showing 'chars' is still confusing with the code point limit, since it's not obvious + // that e.g. combining diacritics or zero-width punctuation count as characters.) + if ( summaryCodePointLimit ) { + mw.widgets.visibleCodePointLimit( wpSummary, summaryCodePointLimit ); + } else if ( summaryByteLimit ) { + mw.widgets.visibleByteLimit( wpSummary, summaryByteLimit ); } // Restore the edit box scroll state following a preview operation, @@ -52,4 +48,4 @@ } ); } } ); -}( mediaWiki, jQuery ) ); +}() );