Author: oliver.buchtala
Description:
Currently, if the menu for a FocusNode gets embedded, this also happens for the Inspector. It would be better to let the Inspector itself decide.
Details:
This gets executed when the menu for a focus node is shown. 'this.embedded' is set according to the size of the focus node:
OTOH, when the inspector gets opened, 'this.embedded' is not recomputed:
This has the consequence that the inspector is rendered over the focussed node.
IMO, at least the same embedded check should be done in the 'inspector' case.
Also the tail should be enabled accordingly.
To leave the inspector implementation even more control I would suggest to add something like this to the Inspector interface:
/**
- @returns {boolean} */
Inspector.prototype.isEmbedded = function() {};
Version: unspecified
Severity: normal