Browse Source

Fix toolbar positioning issues

pull/600/head
Artur Arseniev 9 years ago
parent
commit
3a01f2c72d
  1. 16
      src/commands/view/SelectComponent.js

16
src/commands/view/SelectComponent.js

@ -447,11 +447,8 @@ module.exports = {
} }
this.toolbar.reset(toolbar); this.toolbar.reset(toolbar);
var view = model.view; const view = model.view;
view && this.updateToolbarPos(view.el);
if(view) {
this.updateToolbarPos(view.el);
}
} else { } else {
toolbarStyle.display = 'none'; toolbarStyle.display = 'none';
} }
@ -466,6 +463,7 @@ module.exports = {
var unit = 'px'; var unit = 'px';
var toolbarEl = this.canvas.getToolbarEl(); var toolbarEl = this.canvas.getToolbarEl();
var toolbarStyle = toolbarEl.style; var toolbarStyle = toolbarEl.style;
toolbarStyle.display = 'block';
var pos = this.canvas.getTargetToElementDim(toolbarEl, el, { var pos = this.canvas.getTargetToElementDim(toolbarEl, el, {
elPos, elPos,
event: 'toolbarPosUpdate', event: 'toolbarPosUpdate',
@ -473,6 +471,7 @@ module.exports = {
var leftPos = pos.left + pos.elementWidth - pos.targetWidth; var leftPos = pos.left + pos.elementWidth - pos.targetWidth;
toolbarStyle.top = pos.top + unit; toolbarStyle.top = pos.top + unit;
toolbarStyle.left = leftPos + unit; toolbarStyle.left = leftPos + unit;
toolbarStyle.display = '';
}, },
/** /**
@ -522,9 +521,10 @@ module.exports = {
* Update attached elements, eg. component toolbar * Update attached elements, eg. component toolbar
* @return {[type]} [description] * @return {[type]} [description]
*/ */
updateAttached() { updateAttached(updated) {
var model = this.em.get('selectedComponent'); const model = this.em.get('selectedComponent');
if (model) {
if (model && updated === model) {
var view = model.view; var view = model.view;
this.updateToolbarPos(view.el); this.updateToolbarPos(view.el);
this.showFixedElementOffset(view.el); this.showFixedElementOffset(view.el);

Loading…
Cancel
Save