Last modified: 2014-05-15 11:51:39 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T53415, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 51415 - VisualEditor: Link inspector ignores typed text if user presses [enter] too quickly, explodes with new links
VisualEditor: Link inspector ignores typed text if user presses [enter] too q...
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
Editing Tools (Other open bugs)
unspecified
All All
: High major
: VE-deploy-2013-08-15
Assigned To: Rob Moen
:
: 52344 (view as bug list)
Depends on:
Blocks: ve-linkediting
  Show dependency treegraph
 
Reported: 2013-07-16 01:09 UTC by Erik Moeller
Modified: 2014-05-15 11:51 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Erik Moeller 2013-07-16 01:09:17 UTC
The link inspector ignores the text entered when the user does not wait for the "matching page" results to appear (Chrome/Ubuntu) before pressing "enter". If the user is adding a completely new link, the selection appears offscreen and we get a JavaScript error:

Uncaught Error: No class registered by that name: undefined 

This behavior is similar to earlier issues such as bug 42278 and bug 49941.
Comment 1 Gerrit Notification Bot 2013-07-30 22:06:45 UTC
Change 76839 had a related patch set uploaded by Trevor Parscal:
The greatest commit in the history of the world*

https://gerrit.wikimedia.org/r/76839
Comment 2 Bartosz Dziewoński 2013-07-31 20:41:49 UTC
*** Bug 52344 has been marked as a duplicate of this bug. ***
Comment 3 Gerrit Notification Bot 2013-07-31 20:54:58 UTC
Change 76839 merged by jenkins-bot:
Link inspector bug fixes

https://gerrit.wikimedia.org/r/76839
Comment 4 James Forrester 2013-07-31 21:08:59 UTC
This is now merged and will be deployed later.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links