r98753 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r98752‎ | r98753 | r98754 >
Date:12:57, 3 October 2011
Author:reedy
Status:ok
Tags:
Comment:
Modified paths:
  • /branches/wmf/1.18wmf1/extensions/ProofreadPage/proofread.js (modified) (history)
  • /branches/wmf/1.18wmf1/extensions/WikiEditor/WikiEditor.php (modified) (history)
  • /branches/wmf/1.18wmf1/extensions/WikiEditor/modules/ext.wikiEditor.toolbar.hideSig.js (modified) (history)

Diff [purge]

Index: branches/wmf/1.18wmf1/extensions/WikiEditor/WikiEditor.php
@@ -504,8 +504,5 @@
505505 ),
506506 'ext.wikiEditor.toolbar.hideSig' => $wikiEditorTpl + array(
507507 'scripts' => 'ext.wikiEditor.toolbar.hideSig.js',
508 - 'dependencies' => array(
509 - 'ext.wikiEditor',
510 - )
511508 ),
512509 );
Index: branches/wmf/1.18wmf1/extensions/WikiEditor/modules/ext.wikiEditor.toolbar.hideSig.js
@@ -2,7 +2,10 @@
33 * Remove the signature button if the main namespace is edited.
44 */
55 $( document ).ready( function() {
6 - if ( !$.wikiEditor.isSupported( $.wikiEditor.modules.toolbar ) ) {
 6+ // This module is designed not to depend on ext.wikiEditor or jquery.wikiEditor.
 7+ // Removing this dependency fixed various bugs, but it does mean that we have to
 8+ // account for the situation where $.wikiEditor is not present
 9+ if ( !$.wikiEditor || !$.wikiEditor.isSupported( $.wikiEditor.modules.toolbar ) ) {
710 return;
811 }
912 if ( $( 'body' ).hasClass( 'ns-0' ) ) {
Index: branches/wmf/1.18wmf1/extensions/ProofreadPage/proofread.js
@@ -126,15 +126,15 @@
127127 '<div id="prp_header" style="">' +
128128 '<span style="color:gray;font-size:80%;line-height:100%;">' +
129129 escapeQuotesHTML( mediaWiki.msg( 'proofreadpage_header' ) ) + '</span>' +
130 - '<textarea name="wpHeaderTextbox" rows="2" cols="80" tabindex=1>' + escapeQuotesHTML( pageHeader ) + '</textarea><br />' +
 130+ '<textarea name="wpHeaderTextbox" rows="2" cols="80" tabindex=1>\n' + escapeQuotesHTML( pageHeader ) + '</textarea><br />' +
131131 '<span style="color:gray;font-size:80%;line-height:100%;">' +
132132 escapeQuotesHTML( mediaWiki.msg( 'proofreadpage_body' ) ) + '</span></div>' +
133 - '<textarea name="wpTextbox1" id="wpTextbox1" tabindex=1 style="height:' + ( self.DisplayHeight - 6 ) + 'px;">' +
 133+ '<textarea name="wpTextbox1" id="wpTextbox1" tabindex=1 style="height:' + ( self.DisplayHeight - 6 ) + 'px;">\n' +
134134 escapeQuotesHTML( pageBody ) + '</textarea>' +
135135 '<div id="prp_footer" style="">' +
136136 '<span style="color:gray;font-size:80%;line-height:100%;">' +
137137 escapeQuotesHTML( mediaWiki.msg( 'proofreadpage_footer' ) ) + '</span><br />' +
138 - '<textarea name="wpFooterTextbox" rows="2" cols="80" tabindex=1>' +
 138+ '<textarea name="wpFooterTextbox" rows="2" cols="80" tabindex=1>\n' +
139139 escapeQuotesHTML( pageFooter ) + '</textarea></div>';
140140 }
141141

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r98432WikiEditor: Fix bug where ext.wikiEditor would load and wrap the textarea eve...catrope19:04, 29 September 2011
r98583* (bug 26028) Preserve initial newlines in ProofreadPage customized edit inte...brion23:34, 30 September 2011

Status & tagging log