Krinkle has uploaded a new change for review.
https://gerrit.wikimedia.org/r/71159
Change subject: mw.ViewPageTarget: Support for non-paragraphs in #showWarning
......................................................................
mw.ViewPageTarget: Support for non-paragraphs in #showWarning
We need this for captcha's. They'll be added and displayed as
a warning, and cleared like all others. But they shouldn't be
wrapped in a paragraph and aren't delivered as html text but
as a jQuery object (can't serialise to static html since we'll
have to keep a reference to the TextInputWidget and what not).
Change-Id: I3734fbd3c6f0270094db014bbc76d065a73ab918
---
M modules/ve/init/mw/targets/ve.init.mw.ViewPageTarget.js
1 file changed, 16 insertions(+), 9 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/VisualEditor
refs/changes/59/71159/1
diff --git a/modules/ve/init/mw/targets/ve.init.mw.ViewPageTarget.js
b/modules/ve/init/mw/targets/ve.init.mw.ViewPageTarget.js
index a38c57e..22fe047 100644
--- a/modules/ve/init/mw/targets/ve.init.mw.ViewPageTarget.js
+++ b/modules/ve/init/mw/targets/ve.init.mw.ViewPageTarget.js
@@ -2074,17 +2074,24 @@
/**
* Show an inline warning.
* @param {string} name Warning's unique name
- * @param {string} messageHtml Warning message HTML
+ * @param {string|jQuery} message Warning message (string of HTML, not text,
or jQuery object)
+ * @param {Object} [options]
+ * @param {boolean} [options.wrap=true] Wrap the message in a paragraph.
*/
-ve.init.mw.ViewPageTarget.prototype.showWarning = function ( name, messageHtml
) {
+ve.init.mw.ViewPageTarget.prototype.showWarning = function ( name, message,
options ) {
+ var node;
if ( !this.warnings[name] ) {
- var warning = $(
- '<p
class="ve-init-mw-viewPageTarget-saveDialog-warning">' + messageHtml + '</p>'
- );
- this.$saveDialog
- .find( '.ve-init-mw-viewPageTarget-saveDialog-warnings'
)
- .append( warning );
- this.warnings[name] = warning;
+ options = options || {};
+ node = $( '<div
class="ve-init-mw-viewPageTarget-saveDialog-warning"></div>' );
+ if ( options.wrap !== false ) {
+ node.append( $( '<p>').append( message ) );
+ } else {
+ node.append( message );
+ }
+ this.$saveDialog.find(
'.ve-init-mw-viewPageTarget-saveDialog-warnings' )
+ .append( node );
+
+ this.warnings[name] = node;
}
};
--
To view, visit https://gerrit.wikimedia.org/r/71159
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I3734fbd3c6f0270094db014bbc76d065a73ab918
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Krinkle <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
