mirror of
https://github.com/silverstripe/silverstripe-framework
synced 2024-10-22 12:05:37 +00:00
ENHANCEMENT Simplified "insert link" interface and logic by reverting to TinyMCE's standard behaviour: Only allow linking on an existing selection, don't allow modifying the link content in the "insert" interface. Now that we're presenting this interface as a dialog rather than persistent sidebar its no longer necessary to have "unselected link creation+insertion".
This commit is contained in:
Ingo Schommer
parent
6bafc76118
commit
98852677d5
@ -265,7 +265,6 @@ class HtmlEditorField_Toolbar extends RequestHandler {
|
||||
new EmailField('email', _t('HtmlEditorField.EMAIL', 'Email address')),
|
||||
new TreeDropdownField('file', _t('HtmlEditorField.FILE', 'File'), 'File', 'Filename', 'Title', true),
|
||||
new TextField('Anchor', _t('HtmlEditorField.ANCHORVALUE', 'Anchor')),
|
||||
new TextField('LinkText', _t('HtmlEditorField.LINKTEXT', 'Link text')),
|
||||
new TextField('Description', _t('HtmlEditorField.LINKDESCR', 'Link description')),
|
||||
new CheckboxField('TargetBlank', _t('HtmlEditorField.LINKOPENNEWWIN', 'Open link in a new window?')),
|
||||
new HiddenField('Locale', null, $this->controller->Locale)
|
||||
|
||||
@ -86,6 +86,9 @@
|
||||
$.entwine('ss', function($) {
|
||||
|
||||
$('form.htmleditorfield-form').entwine({
|
||||
|
||||
Bookmark: null,
|
||||
|
||||
onmatch: function() {
|
||||
// Move title from headline to (jQuery compatible) title attribute
|
||||
var titleEl = this.find(':header:first');
|
||||
@ -103,10 +106,12 @@
|
||||
},
|
||||
close: function() {
|
||||
this.dialog('close');
|
||||
this.setBookmark(null);
|
||||
},
|
||||
open: function() {
|
||||
this.dialog('open');
|
||||
this.redraw();
|
||||
this.setBookmark(this.getEditor().selection.getBookmark());
|
||||
},
|
||||
getEditor: function() {
|
||||
return tinyMCE.activeEditor;
|
||||
@ -115,8 +120,6 @@
|
||||
|
||||
$('form.htmleditorfield-linkform').entwine({
|
||||
|
||||
OriginalSelection: null,
|
||||
|
||||
onmatch: function() {
|
||||
this._super();
|
||||
|
||||
@ -131,7 +134,7 @@
|
||||
this.respondToNodeChange();
|
||||
this.dialog('open');
|
||||
this.redraw();
|
||||
this.setOriginalSelection(null);
|
||||
this.setBookmark(this.getEditor().selection.getBookmark());
|
||||
},
|
||||
|
||||
close: function() {
|
||||
@ -176,7 +179,7 @@
|
||||
},
|
||||
|
||||
insertLink: function() {
|
||||
var href, target = null, anchor = this.find(':input[name=Anchor]').val();
|
||||
var href, target = null, anchor = this.find(':input[name=Anchor]').val(), ed = this.getEditor();
|
||||
|
||||
// Determine target
|
||||
if(this.find(':input[name=TargetBlank]').is(':checked')) target = '_blank';
|
||||
@ -210,65 +213,15 @@
|
||||
break;
|
||||
}
|
||||
|
||||
if(this.getOriginalSelection()) {
|
||||
tinyMCE.activeEditor.selection.setRng(this.getOriginalSelection());
|
||||
}
|
||||
|
||||
var linkText = this.find(':input[name=LinkText]').val();
|
||||
var attributes = {
|
||||
href : href,
|
||||
target : target,
|
||||
title : this.find(':input[name=Description]').val(),
|
||||
innerHTML : linkText ? linkText : "Your Link"
|
||||
title : this.find(':input[name=Description]').val()
|
||||
};
|
||||
|
||||
// Add the new link
|
||||
this._insertLink(attributes);
|
||||
},
|
||||
|
||||
/**
|
||||
* Insert a link into the given editor.
|
||||
* Replaces mceInsertLink in that innerHTML can also be set
|
||||
*/
|
||||
_insertLink: function(attributes) {
|
||||
var ed = this.getEditor(), v = attributes,
|
||||
s = ed.selection, e = ed.dom.getParent(s.getNode(), 'A');
|
||||
|
||||
if(tinymce.is(attributes, 'string')) attributes = {href : attributes};
|
||||
|
||||
function set(e) {
|
||||
tinymce.each(attributes, function(v, k) {
|
||||
if(k == 'innerHTML') e.innerHTML = v;
|
||||
else ed.dom.setAttrib(e, k, v);
|
||||
});
|
||||
try {
|
||||
s.select(e);
|
||||
this.updateSelection();
|
||||
} catch(er) {}
|
||||
};
|
||||
|
||||
function replace() {
|
||||
tinymce.each(ed.dom.select('a'), function(e) {
|
||||
if (e.href == 'javascript:mctmp(0);') set(e);
|
||||
});
|
||||
}
|
||||
|
||||
if(attributes.innerHTML && !ed.selection.getContent()) {
|
||||
if(tinymce.isIE) var rng = ed.selection.getRng();
|
||||
e = ed.getDoc().createElement('a');
|
||||
e.href = 'javascript:mctmp(0);';
|
||||
s.setNode(e);
|
||||
if(tinymce.isIE) tinyMCE.activeEditor.selection.setRng(rng);
|
||||
replace();
|
||||
}
|
||||
|
||||
if (!e) {
|
||||
ed.execCommand('CreateLink', false, 'javascript:mctmp(0);');
|
||||
replace();
|
||||
} else {
|
||||
if (attributes.href) set(e);
|
||||
else ed.dom.remove(e, 1);
|
||||
}
|
||||
ed.selection.moveToBookmark(this.getBookmark());
|
||||
ed.execCommand("mceInsertLink", false, attributes);
|
||||
|
||||
this.trigger('onafterinsert', attributes);
|
||||
|
||||
@ -280,11 +233,6 @@
|
||||
this.close();
|
||||
},
|
||||
|
||||
updateSelection: function() {
|
||||
var ed = this.getEditor();
|
||||
if(ed.selection.getRng()) this.setOriginalSelection(ed.selection.getRng());
|
||||
},
|
||||
|
||||
addAnchorSelector: function() {
|
||||
// Avoid adding twice
|
||||
if(this.find(':input[name=AnchorSelector]').length) return;
|
||||
@ -323,7 +271,7 @@
|
||||
var selector = this.find(':input[name=AnchorSelector]'), anchors = new Array();
|
||||
// name attribute is defined as CDATA, should accept all characters and entities
|
||||
// http://www.w3.org/TR/1999/REC-html401-19991224/struct/links.html#h-12.2
|
||||
var raw = tinyMCE.activeEditor.getContent().match(/name="([^"]+?)"|name='([^']+?)'/gim);
|
||||
var raw = this.getEditor().getContent().match(/name="([^"]+?)"|name='([^']+?)'/gim);
|
||||
if (raw && raw.length) {
|
||||
for(var i = 0; i < raw.length; i++) {
|
||||
anchors.push(raw[i].substr(6).replace(/"$/, ''));
|
||||
@ -402,8 +350,6 @@
|
||||
href = RegExp.$1;
|
||||
}
|
||||
|
||||
var linkText = ed.selection.getContent({format : 'html'}).replace(/<\/?a[^>]*>/ig,'');
|
||||
|
||||
// Get rid of TinyMCE's temporary URLs
|
||||
if(href.match(/^javascript:\s*mctmp/)) href = '';
|
||||
|
||||
@ -411,30 +357,26 @@
|
||||
return {
|
||||
LinkType: 'email',
|
||||
email: RegExp.$1,
|
||||
LinkText: linkText,
|
||||
Description: title
|
||||
}
|
||||
} else if(href.match(/^(assets\/.*)$/)) {
|
||||
return {
|
||||
LinkType: 'file',
|
||||
file: RegExp.$1,
|
||||
LinkText: linkText,
|
||||
Description: title
|
||||
}
|
||||
} else if(href.match(/^#(.*)$/)) {
|
||||
return {
|
||||
LinkType: 'anchor',
|
||||
Anchor: RegExp.$1,
|
||||
LinkText: linkText,
|
||||
Description: title,
|
||||
TargetBlank: target ? true : false
|
||||
}
|
||||
} else if(href.match(/^\[sitetree_link id=([0-9]+)\]?(#.*)?$/)) {
|
||||
} else if(href.match(/^\[sitetree_link\s*(?:%20)?id=([0-9]+)\]?(#.*)?$/)) {
|
||||
return {
|
||||
LinkType: 'internal',
|
||||
internal: RegExp.$1,
|
||||
Anchor: RegExp.$2 ? RegExp.$2.substr(1) : '',
|
||||
LinkText: linkText,
|
||||
Description: title,
|
||||
TargetBlank: target ? true : false
|
||||
}
|
||||
@ -442,14 +384,12 @@
|
||||
return {
|
||||
LinkType: 'external',
|
||||
external: href,
|
||||
LinkText: linkText,
|
||||
Description: title,
|
||||
TargetBlank: target ? true : false
|
||||
}
|
||||
} else {
|
||||
return {
|
||||
LinkType: 'internal',
|
||||
LinkText: linkText
|
||||
LinkType: 'internal'
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user