New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[studio-ui] Cannot see/insert layout when RTE setup contains a single layout #1690

Closed
russdanner opened this Issue Dec 13, 2017 · 2 comments

Comments

@russdanner
Member

russdanner commented Dec 13, 2017

Applies to 2.x, 3.x

Expected behavior

Single layout configuration should produce dropdown in RTE toolbar

Actual behavior

No layout option shows up
Error appears in browser console:

Steps to reproduce the problem

  • Create a rte setup having exactly one layout
  • Edit content using RTE with that setup
  • Click on "Insert Layout" button

Log/stack trace (use https://gist.github.com)

Uncaught TypeError: Cannot read property 'prototype' of undefined 
at i.(anonymous function).<anonymous> (https://HOST/studio/static-assets/components/cstudio-forms/controls/rte-plugins/insert-layout.js?version=e75690a6e64131f05862d204ca2a9fe63beca385:29:52) 
at Dispatcher.dispatch (https://HOST/studio/static-assets/modules/editors/tiny_mce/tiny_mce.js?version=e75690a6e64131f05862d204ca2a9fe63beca385:1:6000) 
at i.(anonymous function).tinymce.ui.MenuButton.renderMenu (https://HOST/studio/static-assets/components/cstudio-forms/controls/rte.js?version=e75690a6e64131f05862d204ca2a9fe63beca385:1112:20) 
at i.(anonymous function).tinymce.ui.MenuButton.showMenu (https://HOST/studio/static-assets/components/cstudio-forms/controls/rte.js?version=e75690a6e64131f05862d204ca2a9fe63beca385:1127:11) 
at HTMLAnchorElement.<anonymous> (https://HOST/studio/static-assets/modules/editors/tiny_mce/tiny_mce.js?version=e75690a6e64131f05862d204ca2a9fe63beca385:1:147760) 
at j (https://HOST/studio/static-assets/modules/editors/tiny_mce/tiny_mce.js?version=e75690a6e64131f05862d204ca2a9fe63beca385:1:58169) 
at HTMLAnchorElement.y (https://HOST/studio/static-assets/modules/editors/tiny_mce/tiny_mce.js?version=e75690a6e64131f05862d204ca2a9fe63beca385:1:58327)

Specs: version, OS, browser, etc.

Chrome/Firefox

@sumerjabri sumerjabri added this to Backlog in Crafter CMS v3.0.x via automation Dec 13, 2017

@sumerjabri sumerjabri added this to the Crafter CMS v3.0.4 milestone Dec 13, 2017

@vestrada vestrada assigned vestrada and unassigned jvega190 Dec 14, 2017

@vestrada vestrada moved this from Backlog to In Progress in Crafter CMS v3.0.x Dec 14, 2017

vestrada added a commit to vestrada/studio-ui that referenced this issue Dec 14, 2017

craftercms/craftercms#1690 - [studio-ui] Cannot see/insert layout whe…
…n RTE setup contains a single layout #1690

@vestrada vestrada referenced this issue Dec 14, 2017

Merged

#1690 #722

vestrada added a commit to vestrada/studio-ui that referenced this issue Dec 14, 2017

craftercms/craftercms#1690 - [studio-ui] Cannot see/insert layout whe…
…n RTE setup contains a single layout #1690

@vestrada vestrada referenced this issue Dec 14, 2017

Merged

#1690 #723

@vestrada

This comment has been minimized.

Show comment
Hide comment
@vestrada
Member

vestrada commented Dec 14, 2017

@vestrada vestrada moved this from In Progress to Test & Validate in Crafter CMS v3.0.x Dec 14, 2017

@vestrada vestrada assigned russdanner and unassigned vestrada Dec 14, 2017

@russdanner

This comment has been minimized.

Show comment
Hide comment
@russdanner

russdanner Jan 12, 2018

Member

CONFIRMED

Member

russdanner commented Jan 12, 2018

CONFIRMED

@russdanner russdanner closed this Jan 12, 2018

Crafter CMS v3.0.x automation moved this from Test & Validate to Completed Jan 12, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment