Last modified: 2014-10-20 18:37:01 UTC
Steps to reproduce: 1.Open a page with VE 2.Go to Insert>Formula 3.Attempt to insert "\pi" inside the inspector Observed Result: As soon as you type \ in the inspector , it closes and tries to parse that \ and shows error "Failed to parse (lexing error)" Environment:Betalabs Browser:Chrome
It only closes when you're inserting a new formula. Editing existing ones doesn't have this problem.
Focus issue?
After dm.Surface emits 'contextChange', ve.ui.Context#afterContextChange thinks that the inspector should be closed, because `selectedNode !== this.lastSelectedNode` (this.lastSelectedNode is null). I spent way too much time debugging this and I still don't know why that happens. It worries me somewhat that `selectedNode` is a paragraph node rather than mwMath node.
(The cause is probably 49a239b4b68719dc59bacf79e5186aceca881c58.)
Change 162473 had a related patch set uploaded by Bartosz Dziewoński: ve.ui.Context: Prevent context changes during opening/closing inspectors better https://gerrit.wikimedia.org/r/162473
Change 162495 had a related patch set uploaded by Esanders: Calculate selected node on change and emit contextChange https://gerrit.wikimedia.org/r/162495
Change 162473 abandoned by Bartosz Dziewoński: ve.ui.Context: Prevent context changes during opening/closing inspectors better Reason: Ed probably knows better. https://gerrit.wikimedia.org/r/162473
Change 162495 merged by jenkins-bot: Calculate selected node on change and emit contextChange https://gerrit.wikimedia.org/r/162495
Verified the fix in Betalabs
Verified the fix in test2
Verified the fix in production
This started happening within last couple of hours
But the difference is, this time it does not try to parse each time a character is inserted , just closes the inspector
*** Bug 72136 has been marked as a duplicate of this bug. ***
Moving back to "FIXED" and re-using bug 72136.
Verified the fix in test2 and Betalabs
