Last modified: 2012-12-31 17:24:08 UTC
MediaWiki 1.20 / 07b8a42dddab0bb1cbfd0cd933929bbcc2170d54 Cannot edit edit summary field in Firefox13 and Chrome 21.0.1180.15 dev/OSX. Antoine is bisecting.
Issue is introduced by Gerrit change #12713 "jquery.byteLimit: Partial rewrite to fix logic errors"
Testswarm job for 12713 https://integration.mediawiki.org/testswarm/job/3592/
Will probably be reverted (Gerrit change #14175), so lowering priority.
Debugging in Chrome, looks like it is not caused by jquery.byteLimit. The browser itself is limiting it. The native maxLength property is being set to 0. Turns out that if you set Element.maxLength = undefined; (implied by $input.removeProp('maxLength');) The property becomes undefined indeed in Chrome and Firefox. But the attribute value is changed to maxlength="0". So the very code that was indented to disable te native maxLength ends up enforcing it even stronger. Will try to figure out a different solution then, and file a bug upstream.
Revert has been merged. I filed bug 38163 for the bug that I tried to solve in that commit.
That revert still did not fix the issue - the property is still set to undefined (resulting in zero) when there is a modifier function (Reported at http://de.wikipedia.org/?oldid=107992278#jquery.byteLimit_und_sein_Callback). Gerrit change #23541 should solve that by fully removing the attribute.
Fixed in I9c204243d0bdf7158aa86a62b591ce717a36fe27.
This is not fixed, as neither patch is merged.
RobLa, since this is a pretty major problem for Firefox 13 & Chrome users (they simply cannot edit the edit summary box), it seems like this should block the next deployment. MatmaRex suggests we should merge the quick-fix changeset at https://gerrit.wikimedia.org/r/#/c/23541/ while waiting for Krinkle to improve the more systematic fix.
(In reply to comment #9) > while waiting for Krinkle to improve the more systematic fix. Done in I9c204243d0bdf7158aa86a62b591ce717a36fe27.
queued for merge to 1.20 tarball.
(In reply to comment #11) > queued for merge to 1.20 tarball. https://gerrit.wikimedia.org/r/25874
Oops, not actually merged yet..
Merged now.
Thanks for nudging on this, Bartosz.
.. and merged to 1.20.
