Page MenuHomePhabricator

VisualEditor: 0,0 initial selection in Firefox when opening a new or pre-existing references means inserting template causes fatal JS error
Closed, ResolvedPublic

Description

Steps to Reproduce:

  1. Open any page in VisualEditor.
  1. Go to Insert > Reference to add a reference. Do NOT click inside the reference box.
  1. Go to Insert > Template to add a template.

Actual Results:
You can't add a template, you can't exit the dialog, you can't do anything except close the browser tab and lose all of your work.

Reproducible: Always

This cannot be reproduced in Safari. This cannot be reproduced if you first click inside the reference box.


Version: unspecified
Severity: major

Details

Reference
bz64434

Event Timeline

Fixed in the Firefox-selection change Ed wrote.

  • Bug 64601 has been marked as a duplicate of this bug. ***

(In reply to Rainer Rillke @commons.wikimedia from comment #4)

Where is the gerrit change?

https://gerrit.wikimedia.org/r/#/c/129806/ makes sure the tools correctly initialize as disabled if the initial selection is null

https://gerrit.wikimedia.org/r/#/c/130262/ makes sure the selection isn't 0,0 the second time you open the reference dialog

Verified the fix in production