https://bugzilla.wikimedia.org/show_bug.cgi?id=36310

--- Comment #11 from Krinkle <[email protected]> 2012-06-02 23:54:30 UTC ---
(In reply to comment #9)
> It is already there (includes/specials/SpecialMovepage.php, around line 290)
> and that makes problems, because the browser is deleted it on first validate,
> that gives -1 for the javascript.
> 
>             <tr>
>                 <td class='mw-label'>" .
>                     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 ) .
>                 "</td>
>             </tr>"
> 
> After the dom is ready and shown, [ .. ]


This is not true. Browsers never delete properties or attributes*. MediaWiki
itself removes the attribute for wikis that are not in HTML5 mode. -1 is the
default value in Firefox when accessing the maxlength property of an element
where that attribute does not exist (look at the raw HTML output. You can see
that MediaWiki removes it before outputting, Firefox doesn't do this).

I refer to comment 5 where I've mentioned 2 things that should be fixed. I
don't think anything else is needed in this case.


-- Krinkle

* Maybe in some other case, but certainly not in this case.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
You are on the CC list for the bug.

_______________________________________________
Wikibugs-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to