Page MenuHomePhabricator

[Regression wmf.8] Saving a page after adding any focusable node throws error "TypeError: rect is undefined"
Closed, ResolvedPublic1 Story Points

Description

Steps to reproduce:
1.Add a graph
2.Save the page

There is an error in the console "TypeError: rect is undefined"

Event Timeline

Ryasmeen created this task.Dec 7 2015, 7:38 PM
Ryasmeen raised the priority of this task from to Needs Triage.
Ryasmeen updated the task description. (Show Details)
Ryasmeen added a project: VisualEditor.
Ryasmeen added a subscriber: Ryasmeen.
Restricted Application added subscribers: StudiesWorld, Aklapper. · View Herald TranscriptDec 7 2015, 7:38 PM
Krenair added a subscriber: Krenair.
Jdforrester-WMF renamed this task from [Regression pre-wmf.8] Saving a page after adding a graph throws error "TypeError: rect is undefined" to [Regression wmf.8] Saving a page after adding a graph throws error "TypeError: rect is undefined".Dec 8 2015, 8:10 PM
Jdforrester-WMF triaged this task as High priority.
Jdforrester-WMF moved this task from To Triage to Blocked on the VisualEditor board.
Jdforrester-WMF set Security to None.
Jdforrester-WMF edited a custom field.
Jdforrester-WMF added a subscriber: ferdbold.
ferdbold moved this task from Backlog to VisualEditor on the Graphs board.Dec 9 2015, 8:59 PM
Ryasmeen renamed this task from [Regression wmf.8] Saving a page after adding a graph throws error "TypeError: rect is undefined" to [Regression wmf.8] Saving a page after adding a graph/media throws error "TypeError: rect is undefined".Dec 10 2015, 8:28 PM

This also happens after adding an image to the page and saving it

Ryasmeen renamed this task from [Regression wmf.8] Saving a page after adding a graph/media throws error "TypeError: rect is undefined" to [Regression wmf.8] Saving a page after adding any focusable node throws error "TypeError: rect is undefined".Dec 10 2015, 8:54 PM

Change 262749 had a related patch set uploaded (by Esanders):
ve.ui.Surface: Account for null viewport

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

Change 262749 merged by jenkins-bot:
ve.ui.Surface: Account for null viewport

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