Skip to content
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

Small bug in TinyMCE's template plugin #100

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Small bug in TinyMCE's template plugin #100

wants to merge 1 commit into from

Commits on Sep 22, 2013

  1. I encountered the following bug:

    Using TinyMCE's template plugin, I tried to insert a snippet where the outmost tag was a DIV with a class:
    
    <div class="ym-grid">
        <div class="ym-g50 ym-gl">
            <div class="ym-gbox">
                <p>Content of the left column</p>
            </div>
        </div>
        <div class="ym-g50 ym-gr">
            <div class="ym-gbox">
                <p>Content of the right column</p>
            </div>
        </div>
    </div>
    
    After adding it, only the outmost DIV was inserted, the other elements just disappeared.
    
    I found out that this happened because my template_selected_content_classes list was empty. In the jscripts\tiny_mce\plugins\template\editor_plugin_src.js there needs to be a test if it is empty before replacing, otherwise every time a construct like this is inserted, the inner HTML will be replaced (with the empty string if nothing is selected in the editor).
    
    IMPORTANT: Since the minified editor_plugin.js is used instead of the editor_plugin_src.js, both files should be renamed so that the editor_plugin_src.js is then the editor_plugin.js. I'll try to submit this fix to the TinyMCE repository as well; hopefully there'll be a fixed minified version, too.
    
    Signed-off-by: Jan-Christoph Ihrens <[email protected]>
    enigmatic-user committed Sep 22, 2013
    Configuration menu
    Copy the full SHA
    f1882fb View commit details
    Browse the repository at this point in the history