Make Special:MovePage's "Reason" field an input, not a textarea
authorMatmaRex <matma.rex@gmail.com>
Fri, 3 May 2013 16:30:05 +0000 (18:30 +0200)
committerMatmaRex <matma.rex@gmail.com>
Fri, 3 May 2013 16:34:31 +0000 (18:34 +0200)
This reverts r45571 (207e3e64), with some whitespace changes.

The original reason for making it a <textarea> was to avoid confusion
with the <input> for new page name. This is invalid now that the page
name field has been split into a namespace dropdown and an actual page
title field.

Bug: 13627
Change-Id: I5ab2092c32682e7a8a1494bf58553971568fa6cf

includes/specials/SpecialMovepage.php

index e7a86a6..e4bce1c 100644 (file)
@@ -293,8 +293,11 @@ class MovePageForm extends UnlistedSpecialPage {
                                        Xml::label( $this->msg( 'movereason' )->text(), 'wpReason' ) .
                                "</td>
                                <td class='mw-input'>" .
-                                       Html::element( 'textarea', array( 'name' => 'wpReason', 'id' => 'wpReason', 'cols' => 60, 'rows' => 2,
-                                       'maxlength' => 200 ), $this->reason ) .
+                                       Xml::input( 'wpReason', 60, $this->reason, array(
+                                               'type' => 'text',
+                                               'id' => 'wpReason',
+                                               'maxlength' => 200,
+                                       ) ) .
                                "</td>
                        </tr>"
                );