2019-11-05 14:07:50 +00:00
|
|
|
var controller = require( 'ext.discussionTools.controller' ),
|
|
|
|
modifier = require( 'ext.discussionTools.modifier' );
|
|
|
|
|
|
|
|
/**
|
|
|
|
* DiscussionTools ReplyWidgetPlain class
|
|
|
|
*
|
|
|
|
* @class mw.dt.ReplyWidgetPlain
|
|
|
|
* @extends mw.dt.ReplyWidget
|
|
|
|
* @constructor
|
|
|
|
* @param {Object} comment Parsed comment object
|
|
|
|
* @param {Object} [config] Configuration options
|
|
|
|
*/
|
|
|
|
function ReplyWidgetPlain( comment, config ) {
|
|
|
|
// Parent constructor
|
|
|
|
ReplyWidgetPlain.super.call( this, comment, config );
|
|
|
|
|
|
|
|
this.mode = 'source';
|
|
|
|
|
|
|
|
// Events
|
|
|
|
this.replyBodyWidget.connect( this, { change: this.onInputChangeThrottled } );
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Inheritance */
|
|
|
|
|
|
|
|
OO.inheritClass( ReplyWidgetPlain, require( 'ext.discussionTools.ReplyWidget' ) );
|
|
|
|
|
|
|
|
/* Methods */
|
|
|
|
|
|
|
|
ReplyWidgetPlain.prototype.createReplyBodyWidget = function ( config ) {
|
|
|
|
return new OO.ui.MultilineTextInputWidget( $.extend( {
|
|
|
|
rows: 3,
|
2020-01-21 19:30:34 +00:00
|
|
|
// TODO: Fix upstream to support a value meaning no max limit (e.g. Infinity)
|
|
|
|
maxRows: 999,
|
2019-11-05 14:07:50 +00:00
|
|
|
autosize: true,
|
|
|
|
// The following classes can be used here:
|
|
|
|
// * mw-editfont-monospace
|
|
|
|
// * mw-editfont-sans-serif
|
|
|
|
// * mw-editfont-serif
|
|
|
|
classes: [ 'mw-editfont-' + mw.user.options.get( 'editfont' ) ]
|
|
|
|
}, config ) );
|
|
|
|
};
|
|
|
|
|
|
|
|
ReplyWidgetPlain.prototype.focus = function () {
|
|
|
|
this.replyBodyWidget.focus();
|
|
|
|
};
|
|
|
|
|
|
|
|
ReplyWidgetPlain.prototype.clear = function () {
|
2019-12-11 04:40:17 +00:00
|
|
|
// Parent method
|
|
|
|
ReplyWidgetPlain.super.prototype.clear.apply( this, arguments );
|
|
|
|
|
2019-11-05 14:07:50 +00:00
|
|
|
this.replyBodyWidget.setValue( '' );
|
|
|
|
};
|
|
|
|
|
|
|
|
ReplyWidgetPlain.prototype.isEmpty = function () {
|
|
|
|
return !this.replyBodyWidget.getValue().trim();
|
|
|
|
};
|
|
|
|
|
2020-02-19 01:25:38 +00:00
|
|
|
ReplyWidgetPlain.prototype.setup = function () {
|
|
|
|
// Parent method
|
|
|
|
ReplyWidgetPlain.super.prototype.setup.call( this );
|
|
|
|
|
|
|
|
this.replyBodyWidget.once( 'change', this.onFirstTransaction.bind( this ) );
|
|
|
|
};
|
|
|
|
|
2019-11-05 14:07:50 +00:00
|
|
|
ReplyWidgetPlain.prototype.onKeyDown = function ( e ) {
|
|
|
|
// Parent method
|
|
|
|
ReplyWidgetPlain.super.prototype.onKeyDown.call( this, e );
|
|
|
|
|
|
|
|
if ( e.which === OO.ui.Keys.ENTER && ( e.ctrlKey || e.metaKey ) ) {
|
|
|
|
this.onReplyClick();
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
};
|
|
|
|
|
|
|
|
ReplyWidgetPlain.prototype.setPending = function ( pending ) {
|
|
|
|
// Parent method
|
|
|
|
ReplyWidgetPlain.super.prototype.setPending.call( this, pending );
|
|
|
|
|
|
|
|
if ( pending ) {
|
|
|
|
this.replyBodyWidget.pushPending();
|
|
|
|
this.replyBodyWidget.setDisabled( true );
|
|
|
|
} else {
|
|
|
|
this.replyBodyWidget.popPending();
|
|
|
|
this.replyBodyWidget.setDisabled( false );
|
|
|
|
}
|
|
|
|
};
|
|
|
|
|
Pick reply insertion point based on parser tree, not DOM tree
I don't like that I had to special-case `<p>` tags (top-level
comments) in this code. I feel like it should be possible to handle
top-level comments and replies in a generic way, but I couldn't find
a way to do it that actually worked.
Notes about changes to the behavior, based on the test cases:
* Given a top-level comment A, if there was a "list gap" in the
replies to it: previously new replies would be incorrectly added at
the location of the gap; now they are added after the last reply.
(T242822)
Example: "pl", comment at "08:23, 29 wrz 2018 (CEST)"
* Given a top-level comment A and a reply to it B that skips an
indentation level: previously new replies to A would be added with
the same indentation level as B; now they are added with the
indentation level of A plus one. (The old behavior wasn't a bug, and
this is an accidental effect of other changes, but it seems okay.)
Example: "pl", comment at "03:22, 30 wrz 2018 (CEST)"
and reply at "09:43, 30 wrz 2018 (CEST)"
* Given a top-level comment A, a reply to it B, and a following
top-level comment C that starts at the same indentation level as B:
previously new replies to A would be incorrectly added in the middle
of the comment C, due to the DOM list structure; now they are added
before C. (T241391)
(It seems that comment C was supposed to be a multi-line reply that
was wrongly indented. Unfortunately we have no way to distinguish
this case from a top-level multi-line comment that just happens to
start with a bullet list.)
Example: "pl", comments at "03:36, 24 paź 2018 (CEST)",
"08:35, 24 paź 2018 (CEST)", "17:14, 24 paź 2018 (CEST)"
* In the "en" example, there are some other changes where funnily
nested tags result in slightly different results with the new code.
They don't look important.
* In rare cases, we must split an existing list to add a reply in the
right place. (Basically add `</ul>` before the reply and `<ul>`
after, but it's a bit awkward in DOM terms.)
Example: split-list.html, comment "aaa"; also split-list2.html
(which is the result of saving the previous reply), comment "aaa"
* The modifier can no longer generate DOM that is invalid HTML, fixing
a FIXME in modifier.test.js (or at least, it doesn't happen in these
test cases any more).
Bug: T241391
Bug: T242822
Change-Id: I2a70db01e9a8916c5636bc59ea8490166966d5ec
2020-01-15 06:09:13 +00:00
|
|
|
ReplyWidgetPlain.prototype.insertNewNodes = function ( newParsoidItem ) {
|
2019-11-05 14:07:50 +00:00
|
|
|
var wikitext = controller.autoSignWikitext( this.getValue() );
|
Pick reply insertion point based on parser tree, not DOM tree
I don't like that I had to special-case `<p>` tags (top-level
comments) in this code. I feel like it should be possible to handle
top-level comments and replies in a generic way, but I couldn't find
a way to do it that actually worked.
Notes about changes to the behavior, based on the test cases:
* Given a top-level comment A, if there was a "list gap" in the
replies to it: previously new replies would be incorrectly added at
the location of the gap; now they are added after the last reply.
(T242822)
Example: "pl", comment at "08:23, 29 wrz 2018 (CEST)"
* Given a top-level comment A and a reply to it B that skips an
indentation level: previously new replies to A would be added with
the same indentation level as B; now they are added with the
indentation level of A plus one. (The old behavior wasn't a bug, and
this is an accidental effect of other changes, but it seems okay.)
Example: "pl", comment at "03:22, 30 wrz 2018 (CEST)"
and reply at "09:43, 30 wrz 2018 (CEST)"
* Given a top-level comment A, a reply to it B, and a following
top-level comment C that starts at the same indentation level as B:
previously new replies to A would be incorrectly added in the middle
of the comment C, due to the DOM list structure; now they are added
before C. (T241391)
(It seems that comment C was supposed to be a multi-line reply that
was wrongly indented. Unfortunately we have no way to distinguish
this case from a top-level multi-line comment that just happens to
start with a bullet list.)
Example: "pl", comments at "03:36, 24 paź 2018 (CEST)",
"08:35, 24 paź 2018 (CEST)", "17:14, 24 paź 2018 (CEST)"
* In the "en" example, there are some other changes where funnily
nested tags result in slightly different results with the new code.
They don't look important.
* In rare cases, we must split an existing list to add a reply in the
right place. (Basically add `</ul>` before the reply and `<ul>`
after, but it's a bit awkward in DOM terms.)
Example: split-list.html, comment "aaa"; also split-list2.html
(which is the result of saving the previous reply), comment "aaa"
* The modifier can no longer generate DOM that is invalid HTML, fixing
a FIXME in modifier.test.js (or at least, it doesn't happen in these
test cases any more).
Bug: T241391
Bug: T242822
Change-Id: I2a70db01e9a8916c5636bc59ea8490166966d5ec
2020-01-15 06:09:13 +00:00
|
|
|
wikitext.split( '\n' ).forEach( function ( line, i ) {
|
|
|
|
if ( i > 0 ) {
|
|
|
|
newParsoidItem = modifier.addSiblingListItem( newParsoidItem );
|
|
|
|
}
|
|
|
|
newParsoidItem.appendChild( modifier.createWikitextNode( line ) );
|
2019-11-05 14:07:50 +00:00
|
|
|
} );
|
|
|
|
};
|
|
|
|
|
|
|
|
ReplyWidgetPlain.prototype.getValue = function () {
|
|
|
|
return this.replyBodyWidget.getValue();
|
|
|
|
};
|
|
|
|
|
|
|
|
module.exports = ReplyWidgetPlain;
|