Merge "Add ime-position-inside ULS class to reply widgets"

This commit is contained in:
jenkins-bot 2020-07-24 22:50:45 +00:00 committed by Gerrit Code Review
commit 377a1159ae
2 changed files with 7 additions and 1 deletions

View file

@ -66,6 +66,9 @@ CommentTargetWidget.prototype.setDocument = function ( docOrHtml ) {
this.getSurface().getView().$element.removeClass( 'mw-body-content' );
this.getSurface().$placeholder.removeClass( 'mw-body-content' );
// Fix jquery.ime position (T255191)
this.getSurface().getView().getDocument().getDocumentNode().$element.addClass( 'ime-position-inside' );
// HACK
this.getSurface().authors = this.authors;
};

View file

@ -22,7 +22,7 @@ OO.inheritClass( ReplyWidgetPlain, require( 'ext.discussionTools.ReplyWidget' )
/* Methods */
ReplyWidgetPlain.prototype.createReplyBodyWidget = function ( config ) {
return new OO.ui.MultilineTextInputWidget( $.extend( {
var textInput = new OO.ui.MultilineTextInputWidget( $.extend( {
rows: 3,
// TODO: Fix upstream to support a value meaning no max limit (e.g. Infinity)
maxRows: 999,
@ -33,6 +33,9 @@ ReplyWidgetPlain.prototype.createReplyBodyWidget = function ( config ) {
// * mw-editfont-serif
classes: [ 'mw-editfont-' + mw.user.options.get( 'editfont' ) ]
}, config ) );
// Fix jquery.ime position (T255191)
textInput.$input.addClass( 'ime-position-inside' );
return textInput;
};
ReplyWidgetPlain.prototype.focus = function () {