From c2df5d4e13e9ab63e8f152d638eec35284efdf2f Mon Sep 17 00:00:00 2001
From: Aleksander Machniak <alec@alec.pl>
Date: Mon, 09 Mar 2015 09:24:05 -0400
Subject: [PATCH] Unified request* event arguments handling, added support for _unlock and _action parameters
---
program/js/editor.js | 243 ++++++++++++++++++++++++++++++++++++-----------
1 files changed, 184 insertions(+), 59 deletions(-)
diff --git a/program/js/editor.js b/program/js/editor.js
index c5680ed..829373a 100644
--- a/program/js/editor.js
+++ b/program/js/editor.js
@@ -36,11 +36,13 @@
function rcube_text_editor(config, id)
{
var ref = this,
+ abs_url = location.href.replace(/[?#].*$/, '').replace(/\/$/, ''),
conf = {
selector: '#' + ($('#' + id).is('.mce_editor') ? id : 'fake-editor-id'),
+ cache_suffix: 's=4010700',
theme: 'modern',
language: config.lang,
- content_css: 'program/js/tinymce/roundcube/content.css?v1',
+ content_css: rcmail.assets_path('program/js/tinymce/roundcube/content.css'),
menubar: false,
statusbar: false,
toolbar_items_size: 'small',
@@ -50,7 +52,8 @@
convert_urls: false, // #1486944
image_description: false,
paste_webkit_style: "color font-size font-family",
- paste_data_images: true
+ paste_data_images: true,
+ browser_spellcheck: true
};
// register spellchecker for plain text editor
@@ -65,10 +68,12 @@
// minimal editor
if (config.mode == 'identity') {
$.extend(conf, {
- plugins: 'autolink charmap code colorpicker hr link paste tabfocus textcolor',
+ plugins: 'autolink charmap code colorpicker hr image link paste tabfocus textcolor',
toolbar: 'bold italic underline alignleft aligncenter alignright alignjustify'
- + ' | outdent indent charmap hr link unlink code forecolor'
- + ' | fontselect fontsizeselect'
+ + ' | outdent indent charmap hr link unlink image code forecolor'
+ + ' | fontselect fontsizeselect',
+ file_browser_callback: function(name, url, type, win) { ref.file_browser_callback(name, url, type); },
+ file_browser_callback_types: 'image'
});
}
// full-featured editor
@@ -79,7 +84,7 @@
toolbar: 'bold italic underline | alignleft aligncenter alignright alignjustify'
+ ' | bullist numlist outdent indent ltr rtl blockquote | forecolor backcolor | fontselect fontsizeselect'
+ ' | link unlink table | emoticons charmap image media | code searchreplace undo redo',
- spellchecker_rpc_url: '../../../../../?_task=utils&_action=spell_html&_remote=1',
+ spellchecker_rpc_url: abs_url + '/?_task=utils&_action=spell_html&_remote=1',
spellchecker_language: rcmail.env.spell_lang,
accessibility_focus: false,
file_browser_callback: function(name, url, type, win) { ref.file_browser_callback(name, url, type); },
@@ -101,6 +106,10 @@
});
ed.on('keypress', function() {
rcmail.compose_type_activity++;
+ });
+ // secure spellchecker requests with Roundcube token
+ tinymce.util.XHR.on('beforeSend', function(e) {
+ e.xhr.setRequestHeader('X-Roundcube-Request', rcmail.env.request_token);
});
};
@@ -144,14 +153,17 @@
window.setTimeout(function() {
window.focus(); // for WebKit (#1486674)
fe.focus();
+ rcmail.env.compose_focus_elem = null;
}, 10);
}
}
- // set tabIndex and set focus to element that was focused before
- this.tabindex(fe && fe.id == this.id);
- // Trigger resize (needed for proper editor resizing in some browsers)
- window.setTimeout(function() { $(window).resize(); }, 100);
+ window.setTimeout(function() {
+ // set tabIndex and set focus to element that was focused before
+ ref.tabindex(fe && fe.id == ref.id);
+ // Trigger resize (needed for proper editor resizing in some browsers)
+ $(window).resize();
+ }, 100);
};
// set tabIndex on tinymce editor
@@ -159,20 +171,19 @@
{
if (rcmail.env.task == 'mail' && this.editor) {
var textarea = this.editor.getElement(),
+ body = this.editor.getBody(),
node = this.editor.getContentAreaContainer().childNodes[0];
if (textarea && node)
node.tabIndex = textarea.tabIndex;
- if (focus)
- this.editor.getBody().focus();
// find :prev and :next elements to get focus when tabbing away
if (textarea.tabIndex > 0) {
var x = null,
- editor = this.editor,
tabfocus_elements = [':prev',':next'],
el = tinymce.DOM.select('*[tabindex='+textarea.tabIndex+']:not(iframe)');
- tinymce.each(el, function(e, i) { if (e.id == editor.id) { x = i; return false; } });
+
+ tinymce.each(el, function(e, i) { if (e.id == ref.id) { x = i; return false; } });
if (x !== null) {
if (el[x-1] && el[x-1].id) {
tabfocus_elements[0] = el[x-1].id;
@@ -180,14 +191,21 @@
if (el[x+1] && el[x+1].id) {
tabfocus_elements[1] = el[x+1].id;
}
- editor.settings.tabfocus_elements = tabfocus_elements.join(',');
+ this.editor.settings.tabfocus_elements = tabfocus_elements.join(',');
}
}
+
+ // ContentEditable reset fixes invisible cursor issue in Firefox < 25
+ if (bw.mz && bw.vendver < 25)
+ $(body).prop('contenteditable', false).prop('contenteditable', true);
+
+ if (focus)
+ body.focus();
}
};
// switch html/plain mode
- this.toggle = function(ishtml)
+ this.toggle = function(ishtml, noconvert)
{
var curr, content, result,
// these non-printable chars are not removed on text2html and html2text
@@ -204,11 +222,12 @@
content = input.val();
// replace current text signature with temp mark
- if (is_sig)
- content = content.replace(signature.text, sig_mark);
+ if (is_sig) {
+ content = content.replace(/\r\n/, "\n");
+ content = content.replace(signature.text.replace(/\r\n/, "\n"), sig_mark);
+ }
- // convert to html
- result = rcmail.plain2html(content, function(data) {
+ var init_editor = function(data) {
// replace signature mark with html version of the signature
if (is_sig)
data = data.replace(sig_mark, '<div id="_rc_sig">' + signature.html + '</div>');
@@ -218,13 +237,25 @@
setTimeout(function() {
if (ref.editor) {
+ var body = $(ref.editor.getBody());
if (rcmail.env.default_font)
- $(ref.editor.getBody()).css('font-family', rcmail.env.default_font);
+ body.css('font-family', rcmail.env.default_font);
// #1486593
ref.tabindex(true);
+ // put cursor on start of the compose body
+ ref.editor.selection.setCursorLocation(body.children().first().get(0));
}
}, 500);
- });
+ };
+
+ // convert to html
+ if (!noconvert) {
+ result = rcmail.plain2html(content, init_editor);
+ }
+ else {
+ init_editor(content);
+ result = true;
+ }
}
else if (this.editor) {
if (is_sig) {
@@ -243,8 +274,7 @@
// get html content
content = this.editor.getContent();
- // convert html to text
- result = rcmail.html2plain(content, function(data) {
+ var init_plaintext = function(data) {
tinymce.execCommand('mceRemoveEditor', false, ref.id);
ref.editor = null;
@@ -253,7 +283,17 @@
data = data.replace(sig_mark, "\n" + signature.text);
input.val(data).focus();
- });
+ rcmail.set_caret_pos(input.get(0), 0);
+ };
+
+ // convert html to text
+ if (!noconvert) {
+ result = rcmail.html2plain(content, init_plaintext);
+ }
+ else {
+ init_plaintext(input.val());
+ result = true;
+ }
// bring back current signature
if (!result && curr)
@@ -281,9 +321,10 @@
var ed = this.editor;
if (ed) {
- if (ed.plugins && ed.plugins.spellchecker && this.spellcheck_active)
- ed.execCommand('mceSpellCheck');
+ if (ed.plugins && ed.plugins.spellchecker && this.spellcheck_active) {
+ ed.execCommand('mceSpellCheck', false);
this.spellcheck_observer();
+ }
}
else if (ed = this.spellchecker) {
if (ed.state && ed.state != 'ready' && ed.state != 'no_error_found')
@@ -302,17 +343,13 @@
return ed.state != 'ready' && ed.state != 'no_error_found';
};
- // resume spellchecking, highlight provided mispellings without new ajax request
+ // resume spellchecking, highlight provided mispellings without a new ajax request
this.spellcheck_resume = function(data)
{
var ed = this.editor;
if (ed) {
- ed.settings.spellchecker_callback = function(name, text, done, error) { done(data); };
- ed.execCommand('mceSpellCheck');
- ed.settings.spellchecker_callback = null;
-
- this.spellcheck_observer();
+ ed.plugins.spellchecker.markErrors(data);
}
else if (ed = this.spellchecker) {
ed.prepare(false, true);
@@ -371,29 +408,33 @@
};
// get selected text (if no selection returns all text) from the editor
- this.get_content = function(selected, plain)
+ this.get_content = function(args)
{
- // apply spellcheck changes if spell checker is active
- this.spellcheck_stop();
+ var sigstart, ed = this.editor, text = '', strip = false,
+ defaults = {refresh: true, selection: false, nosig: false, format: 'html'};
- var sigstart, ed = this.editor,
- format = plain ? 'text' : 'html',
- text = '', strip = false;
+ args = $.extend(defaults, args);
+
+ // apply spellcheck changes if spell checker is active
+ if (args.refresh) {
+ this.spellcheck_stop();
+ }
// get selected text from tinymce editor
if (ed) {
ed.getWin().focus(); // correct focus in IE & Chrome
- if (selected)
- text = ed.selection.getContent({format: format});
+ if (args.selection)
+ text = ed.selection.getContent({format: args.format});
if (!text) {
- text = ed.getContent({format: format});
- strip = true;
+ text = ed.getContent({format: args.format});
+ // @todo: strip signature in html mode
+ strip = args.format == 'text';
}
}
// get selected text from compose textarea
else if (ed = rcube_find_object(this.id)) {
- if (selected && $(ed).is(':focus')) {
+ if (args.selection && $(ed).is(':focus')) {
text = rcmail.get_input_selection(ed).text;
}
@@ -404,7 +445,8 @@
}
// strip off signature
- if (strip) {
+ // @todo: make this optional
+ if (strip && args.nosig) {
sigstart = text.indexOf('-- \n');
if (sigstart > 0) {
text = text.substring(0, sigstart);
@@ -417,7 +459,7 @@
// change user signature text
this.change_signature = function(id, show_sig)
{
- var cursor_pos, p = -1,
+ var position_element, cursor_pos, p = -1,
input_message = $('#' + this.id),
message = input_message.val(),
sig = rcmail.env.identity;
@@ -432,6 +474,7 @@
if (p >= 0)
message = message.substring(0, p) + message.substring(p+sig.length, message.length);
}
+
// add the new signature string
if (show_sig && rcmail.env.signatures && rcmail.env.signatures[id]) {
sig = rcmail.env.signatures[id].text;
@@ -497,12 +540,22 @@
}
else {
body.appendChild(sigElem);
+ position_element = $(sigElem).prev();
}
}
if (rcmail.env.signatures[id]) {
sigElem.innerHTML = rcmail.env.signatures[id].html;
}
+ }
+ else if (!rcmail.env.top_posting) {
+ position_element = $(this.editor.getBody()).children().last();
+ }
+
+ // put cursor before signature and scroll the window
+ if (this.editor && position_element && position_element.length) {
+ this.editor.selection.setCursorLocation(position_element.get(0));
+ this.editor.getWin().scroll(0, position_element.offset().top);
}
};
@@ -523,7 +576,7 @@
// image selector
this.file_browser_callback = function(field_name, url, type)
{
- var i, elem, dialog, list = [];
+ var i, elem, cancel, dialog, fn, list = [];
// open image selector dialog
dialog = this.editor.windowManager.open({
@@ -531,7 +584,7 @@
width: 500,
height: 300,
html: '<div id="image-selector-list"><ul></ul></div>'
- + '<div id="image-selector-form"><div id="image-upload-button" class="mce-widget mce-btn" role="button"></div></div>',
+ + '<div id="image-selector-form"><div id="image-upload-button" class="mce-widget mce-btn" role="button" tabindex="0"></div></div>',
buttons: [{text: 'Cancel', onclick: function() { ref.file_browser_close(); }}]
});
@@ -546,19 +599,55 @@
}
if (list.length) {
- $('#image-selector-list > ul').append(list);
+ $('#image-selector-list > ul').append(list).find('li:first').focus();
}
// add hint about max file size (in dialog footer)
- $('div.mce-abs-end', dialog.getEl()).append($('<div class="hint">').text($('div.hint', rcmail.gui_objects.uploadform).text()));
+ $('div.mce-abs-end', dialog.getEl()).append($('<div class="hint">')
+ .text($('div.hint', rcmail.gui_objects.uploadform).text()));
+
+ // init upload button
+ elem = $('#image-upload-button').append($('<span>').text(rcmail.gettext('add' + type)));
+ cancel = elem.parents('.mce-panel').find('button:last').parent();
+
+ // we need custom Tab key handlers, until we find out why
+ // tabindex do not work here as expected
+ elem.keydown(function(e) {
+ if (e.which == 9) {
+ // on Tab + Shift focus first file
+ if (rcube_event.get_modifier(e) == SHIFT_KEY)
+ $('#image-selector-list li:last').focus();
+ // on Tab focus Cancel button
+ else
+ cancel.focus();
+
+ return false;
+ }
+ });
+ cancel.keydown(function(e) {
+ if (e.which == 9) {
+ // on Tab + Shift focus upload button
+ if (rcube_event.get_modifier(e) == SHIFT_KEY)
+ elem.focus();
+ else
+ $('#image-selector-list li:first').focus();
+
+ return false;
+ }
+ });
// enable (smart) upload button
- elem = $('#image-upload-button').append($('<span>').text(rcmail.gettext('add' + type)));
this.hack_file_input(elem, rcmail.gui_objects.uploadform);
// enable drag-n-drop area
- if (rcmail.gui_objects.filedrop && rcmail.env.filedrop && ((window.XMLHttpRequest && XMLHttpRequest.prototype && XMLHttpRequest.prototype.sendAsBinary) || window.FormData)) {
- rcmail.env.old_file_drop = rcmail.gui_objects.filedrop;
+ if ((window.XMLHttpRequest && XMLHttpRequest.prototype && XMLHttpRequest.prototype.sendAsBinary) || window.FormData) {
+ if (!rcmail.env.filedrop) {
+ rcmail.env.filedrop = {};
+ }
+ if (rcmail.gui_objects.filedrop) {
+ rcmail.env.old_file_drop = rcmail.gui_objects.filedrop;
+ }
+
rcmail.gui_objects.filedrop = $('#image-selector-form');
rcmail.gui_objects.filedrop.addClass('droptarget')
.bind('dragover dragleave', function(e) {
@@ -576,18 +665,25 @@
var elem;
if (elem = ref.file_browser_entry(attr.name, attr.attachment)) {
$('#image-selector-list > ul').prepend(elem);
+ elem.focus();
}
});
}
+
+ // @todo: upload progress indicator
};
// close file browser window
this.file_browser_close = function(url)
{
+ var input = $('#' + rcmail.env.file_browser_field);
+
if (url)
- $('#' + rcmail.env.file_browser_field).val(url);
+ input.val(url);
this.editor.windowManager.close();
+
+ input.focus();
if (rcmail.env.old_file_drop)
rcmail.gui_objects.filedrop = rcmail.env.old_file_drop;
@@ -598,6 +694,10 @@
{
if (!file.complete || !file.mimetype) {
return;
+ }
+
+ if (rcmail.file_upload_id) {
+ rcmail.set_busy(false, null, rcmail.file_upload_id);
}
var rx, img_src;
@@ -617,13 +717,35 @@
}
if (rx.test(file.mimetype)) {
- var href = rcmail.env.comm_path+'&_id='+rcmail.env.compose_id+'&_action=display-attachment&_file='+file_id,
+ var path = rcmail.env.comm_path + '&_from=' + rcmail.env.action,
+ action = rcmail.env.compose_id ? '&_id=' + rcmail.env.compose_id + '&_action=display-attachment' : '&_action=upload-display',
+ href = path + action + '&_file=' + file_id,
img = $('<img>').attr({title: file.name, src: img_src ? img_src : href + '&_thumbnail=1'});
- return $('<li>').data('url', href)
+ return $('<li>').attr({tabindex: 0})
+ .data('url', href)
.append($('<span class="img">').append(img))
.append($('<span class="name">').text(file.name))
- .click(function() { ref.file_browser_close($(this).data('url')); });
+ .click(function() { ref.file_browser_close($(this).data('url')); })
+ .keydown(function(e) {
+ if (e.which == 13) {
+ ref.file_browser_close($(this).data('url'));
+ }
+ // we need custom Tab key handlers, until we find out why
+ // tabindex do not work here as expected
+ else if (e.which == 9) {
+ if (rcube_event.get_modifier(e) == SHIFT_KEY) {
+ if (!$(this).prev().focus().length)
+ $('#image-upload-button').parents('.mce-panel').find('button:last').parent().focus();
+ }
+ else {
+ if (!$(this).next().focus().length)
+ $('#image-upload-button').focus();
+ }
+
+ return false;
+ }
+ });
}
};
@@ -631,7 +753,7 @@
this.hack_file_input = function(elem, clone_form)
{
var link = $(elem),
- file = $('<input>'),
+ file = $('<input>').attr('name', '_file[]'),
form = $('<form>').attr({method: 'post', enctype: 'multipart/form-data'}),
offset = link.offset();
@@ -646,7 +768,7 @@
file.css({top: (e.pageY - offset.top - 10) + 'px', left: (e.pageX - offset.left - 10) + 'px'});
}
- file.attr({type: 'file', multiple: 'multiple', size: 5, title: ''})
+ file.attr({type: 'file', multiple: 'multiple', size: 5, title: '', tabindex: -1})
.change(function() { rcmail.upload_file(form, 'upload'); })
.click(function() { setTimeout(function() { link.mouseleave(); }, 20); })
// opacity:0 does the trick, display/visibility doesn't work
@@ -681,6 +803,9 @@
file.trigger(e);
}
})
+ .keydown(function(e) {
+ if (e.which == 13) file.trigger('click');
+ })
.mouseleave()
.append(form);
};
--
Gitblit v1.9.1