stfalcon / fwdays

web/js/markItUpSettings.js

mySettings = {

The variable mySettings seems to be never declared. Assigning variables without defining them first makes them global. If this was intended, consider making it explicit like using window.mySettings.

    onShiftEnter:	{keepDefault:false, replaceWith:'<br />\n'},
    onCtrlEnter:	{keepDefault:false, openWith:'\n<p>', closeWith:'</p>\n'},
    onTab:			{keepDefault:false, openWith:'	 '},
    markupSet: [
        {name:'Heading 1', key:'1', openWith:'<h1(!( class="[![Class]!]")!)>', closeWith:'</h1>', placeHolder:'Your title here...' },
        {name:'Heading 2', key:'2', openWith:'<h2(!( class="[![Class]!]")!)>', closeWith:'</h2>', placeHolder:'Your title here...' },
        {name:'Heading 3', key:'3', openWith:'<h3(!( class="[![Class]!]")!)>', closeWith:'</h3>', placeHolder:'Your title here...' },
        {name:'Heading 4', key:'4', openWith:'<h4(!( class="[![Class]!]")!)>', closeWith:'</h4>', placeHolder:'Your title here...' },
        {name:'Heading 5', key:'5', openWith:'<h5(!( class="[![Class]!]")!)>', closeWith:'</h5>', placeHolder:'Your title here...' },
        {name:'Heading 6', key:'6', openWith:'<h6(!( class="[![Class]!]")!)>', closeWith:'</h6>', placeHolder:'Your title here...' },
        {name:'Paragraph', openWith:'<p(!( class="[![Class]!]")!)>', closeWith:'</p>' },
        {separator:'---------------' },
        {name:'Bold', key:'B', openWith:'(!(<strong>|!|<b>)!)', closeWith:'(!(</strong>|!|</b>)!)' },
        {name:'Italic', key:'I', openWith:'(!(<em>|!|<i>)!)', closeWith:'(!(</em>|!|</i>)!)' },
        {name:'Stroke through', key:'S', openWith:'<del>', closeWith:'</del>' },
        {separator:'---------------' },
        {name:'Ul', openWith:'<ul>\n', closeWith:'</ul>\n' },
        {name:'Ol', openWith:'<ol>\n', closeWith:'</ol>\n' },
        {name:'Li', openWith:'<li>', closeWith:'</li>' },
        {separator:'---------------' },

        //{name:'PictureSimple', key:'P', replaceWith:'<img src="[![Source:!:http://]!]" alt="[![Alternative text]!]" />' },
        {
            name:'Picture',
            key:'P',
            beforeInsert: function(markItUp) { InlineUpload.display(markItUp) }

The variable InlineUpload seems to be never declared. If this is a global, consider adding a /** global: InlineUpload */ comment.

        },

        {name:'Link', key:'L', openWith:'<a href="[![Link:!:http://]!]"(!( title="[![Title]!]")!)>', closeWith:'</a>', placeHolder:'Your text to link...' },
        {separator:'---------------' },
        {name:'Clean', className:'clean', replaceWith:function(markitup) { return markitup.selection.replace(/<(.*?)>/g, "") } },
        {name:'Preview', className:'preview', call:'preview' },
        {separator:'---------------' },
        {name:'More', className:'mMore', key:'M', openWith:'\n<!--more-->\n'},
        {name:'Code',className:'ins-code-button', key:'D', replaceWith: function(markItUp) { InsCode.display(markItUp) }}

The variable InsCode seems to be never declared. If this is a global, consider adding a /** global: InsCode */ comment.

    ]
};

$(document).ready(function() {
    $("textarea.markitup").markItUp(mySettings);
});