Last modified: 2014-11-15 22:29:53 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 T69530, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 67530 - VisualEditor: ve.ui.FragmentInspector immediately closes after opening on Opera 12, when opened using the context tooltip
VisualEditor: ve.ui.FragmentInspector immediately closes after opening on Ope...
Status: RESOLVED WORKSFORME
Product: VisualEditor
Classification: Unclassified
Editing Tools (Other open bugs)
unspecified
All All
: Low normal
: ---
Assigned To: Editing team bugs – take if you're interested!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-07-04 11:39 UTC by Bartosz Dziewoński
Modified: 2014-11-15 22:29 UTC (History)
5 users (show)

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


Attachments

Description Bartosz Dziewoński 2014-07-04 11:39:55 UTC
The inspector for <syntaxhighlight /> node immediately closes after opening on Opera 12. I don't yet know why, but I am going to find out.
Comment 1 Bartosz Dziewoński 2014-07-04 13:27:43 UTC
When you try opening the inspector:

* The 'focusout' event is eventually fired on .ve-ce-surface-paste
* ve.ce.Surface.prototype.onFocusChange is called
* On Opera 12 `rangy.getSelection( this.getElementDocument() ).anchorNode` is
  null, but on other browsers it returns the .ve-ce-surface-paste element
* Therefore `hasFocus` is false instead of true
* The document gets blurred, which triggers the inspector to hide some ten
  levels of abstraction down.


No idea why the selection gets lost. Perhaps the inspector should reselect?
Comment 2 Bartosz Dziewoński 2014-07-04 13:32:08 UTC
(In reply to Bartosz Dziewoński from comment #1)
> * On Opera 12 `rangy.getSelection( this.getElementDocument() ).anchorNode` is
>   null, but on other browsers it returns the .ve-ce-surface-paste element

Oh, and I forgot the best part: not always. Sometimes the selection is the same as on other browsers. I've somehow caused that several times when debugging, but I have absolutely no idea how.
Comment 3 Bartosz Dziewoński 2014-09-17 14:04:47 UTC
Ooh. It only closes itself when opened by clicking the context tooltip. Opening it by double-clicking the node or by pressing Enter when it's selected keeps the inspector open.

After editing it, when the context tooltip is displayed again, clicking it also will always work correctly. Fascinating.
Comment 4 Bartosz Dziewoński 2014-09-17 14:07:28 UTC
Also affects the comment inspector in standalone.
Comment 5 Bartosz Dziewoński 2014-11-15 22:29:53 UTC
Seems to have been fixed at some point, I can no longer reproduce.

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


Navigation
Links