Browse Source

Test new page creation

pull/3411/head
Artur Arseniev 5 years ago
parent
commit
767d3dc598
  1. 2
      src/canvas/view/CanvasView.js
  2. 3
      src/canvas/view/FrameWrapView.js
  3. 2
      src/editor/model/Editor.js
  4. 2
      src/navigator/index.js
  5. 1
      src/pages/index.js
  6. 14
      test/specs/pages/index.js

2
src/canvas/view/CanvasView.js

@ -347,7 +347,7 @@ export default Backbone.View.extend({
frames.render();
const mainFrame = frms.at(0).view;
em.setCurrentFrame(mainFrame);
framesArea.appendChild(frames.el);
framesArea && framesArea.appendChild(frames.el);
this.frame = mainFrame;
},

3
src/canvas/view/FrameWrapView.js

@ -206,7 +206,8 @@ export default Backbone.View.extend({
`
);
this.elTools = elTools;
cv.toolsWrapper.appendChild(elTools); // TODO remove on frame remove
const twrp = cv.toolsWrapper;
twrp && twrp.appendChild(elTools); // TODO remove on frame remove
onRender &&
onRender({
el,

2
src/editor/model/Editor.js

@ -196,7 +196,7 @@ export default Backbone.Model.extend({
const cfgParent = !isUndefined(config[name])
? config[name]
: config[Mod.name];
const cfg = cfgParent || {};
const cfg = cfgParent === true ? {} : cfgParent || {};
const sm = this.get('StorageManager');
cfg.pStylePrefix = config.pStylePrefix || '';

2
src/navigator/index.js

@ -50,7 +50,7 @@ export default () => {
* @return {self}
*/
setRoot(el) {
layers.setRoot(el);
layers && layers.setRoot(el);
return this;
},

1
src/pages/index.js

@ -60,6 +60,7 @@ export default () => {
*/
add(props, opts = {}) {
props.id = props.id || this._createId();
props.frames = props.frames || [{}];
const page = this.pages.add(props, opts);
opts.select && this.select(page);
return page;

14
test/specs/pages/index.js

@ -8,7 +8,7 @@ describe('Pages', () => {
let pm;
beforeAll(() => {
editor = new Editor();
editor = new Editor({ pageManager: true });
em = editor.getModel();
domc = em.get('DomComponents');
pm = em.get('PageManager');
@ -61,6 +61,18 @@ describe('Pages', () => {
expect(initCmpLen).toBe(1);
});
test('Adding new page with selection', () => {
const name = 'Test page';
const page = pm.add({ name }, { select: 1 });
expect(page.id).toBeTruthy();
expect(page.get('name')).toBe(name);
expect(pm.getSelected()).toBe(page);
expect(pm.getAll().length).toBe(2);
const pageComp = page.getMainComponent();
expect(pageComp.is('wrapper')).toBe(true);
expect(pageComp.components().length).toBe(0);
});
// describe.only('Enable page manager', () => {});
describe.skip('Init with pages', () => {

Loading…
Cancel
Save