Page MenuHomePhabricator

Edit summary length counter and popup disagree
Closed, ResolvedPublic1 Estimated Story Points

Description

According to the little number I have 43 characters left, according to the popup I already have 12 too many:

2017-06-18_edit summary count.png (120×710 px, 16 KB)

Add mention of interwiki search so that users find out about it from the help link, after [[mailarchive:discovery/2017-June/001539.html|[discovery] Search update: sister project snippets are now in production!]].

Event Timeline

I can reproduce in Firefox (though there instead of a popup I just get a red border; and - even worse - I can't submit the form).
The input has maxlength="200" set (for non-JS-clients). This limit should be removed (or increased to 255) by jquery.byteLimit.js, but apparently something is broken in the OOUI design. Probably T168042: Infusing EditPage's OOUIs widget resets the accesskey in tooltips (conflict with jquery.accessKeyLabel) is related, note that the tooltip has no prefix for the accesskey as described there, and probably the same code that reset the title attribute also reset the maxlength.

Probably caused by 5e4b02c or one of it follow-ups. $( '#wpSummary' ).byteLimit( 255 ); happens before infusing there, so is likely to be (partially) overridden by the infusion, causing the original limit to come into effect again.

matmarex triaged this task as Medium priority.

Change 359980 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[mediawiki/core@master] mw.action.edit: Fix byte limit for the edit summary field

https://gerrit.wikimedia.org/r/359980

Change 359980 merged by jenkins-bot:
[mediawiki/core@master] mw.action.edit: Fix byte limit for the edit summary field

https://gerrit.wikimedia.org/r/359980

Jdforrester-WMF edited projects, added Editing-team; removed Patch-For-Review.
Jdforrester-WMF set the point value for this task to 1.

Aha, had noticed this yesterday, thanks for the fix!