Last modified: 2014-10-23 21:55:47 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 T74095, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 72095 - VisualEditor: [Regression pre-wmf4] After placing the cursor on a slug, placing it anywhere in the page which follows that particular slug throws "TypeError: Cannot read property 'getSelection' of null"
VisualEditor: [Regression pre-wmf4] After placing the cursor on a slug, placi...
Status: VERIFIED FIXED
Product: VisualEditor
Classification: Unclassified
Editing Tools (Other open bugs)
unspecified
All All
: High normal
: VE-deploy-2014-10-16
Assigned To: Roan Kattouw
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-10-15 20:47 UTC by Rummana Yasmeen
Modified: 2014-10-23 21:55 UTC (History)
6 users (show)

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


Attachments
Screenshot (67.63 KB, image/png)
2014-10-15 20:47 UTC, Rummana Yasmeen
Details

Description Rummana Yasmeen 2014-10-15 20:47:37 UTC
Created attachment 16774 [details]
Screenshot

Steps to reproduce:

1.Open a page with VE
2.Insert a bullet or numbered list
3.Keep on adding indentation to the list to create slugs
4.Place your cursor inside the slug to add an item to the list
Observed Result:
It throws error in the console:TypeError: Cannot read property 'getSelection' of null

See the screenshot attached
Comment 1 Rummana Yasmeen 2014-10-15 20:53:12 UTC
Rephrasing the bug its not only for slugs created from the list but rather a generic issue for all kind of slugs.It happens if you place your cursor in a slug and then place your cursor anywhere in the page which follows the slug.
If you place your cursor somehwere above the slug , you wont get the error.
Comment 2 Gerrit Notification Bot 2014-10-15 21:23:24 UTC
Change 166884 had a related patch set uploaded by Catrope:
Followup 2432cbf: add recursion guard to updateSlug

https://gerrit.wikimedia.org/r/166884
Comment 3 Gerrit Notification Bot 2014-10-15 21:59:39 UTC
Change 166884 merged by jenkins-bot:
Followup 2432cbf: add recursion guard to updateSlug

https://gerrit.wikimedia.org/r/166884
Comment 4 Rummana Yasmeen 2014-10-16 01:34:33 UTC
Verified the fix in Betalabs
Comment 5 Rummana Yasmeen 2014-10-17 00:01:52 UTC
Verified the fix in test2
Comment 6 Rummana Yasmeen 2014-10-23 21:55:12 UTC
Verified the fix in production

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


Navigation
Links