Skip to content
This repository has been archived by the owner on Nov 9, 2017. It is now read-only.

Double Click #28

Open
tru10000 opened this issue Oct 27, 2010 · 1 comment
Open

Double Click #28

tru10000 opened this issue Oct 27, 2010 · 1 comment

Comments

@tru10000
Copy link

Small small typo on line 247 in MooEditable.js:
dbllick: this.editorDoubleClick.bind(this),
should be:
dblclick: this.editorDoubleClick.bind(this),

Also would you be interested in expanding

editorDoubleClick: function(e){
    this.fireEvent('editorDoubleClick', [e, this]);
},

to
editorDoubleClick: function(e){
this.actions.each(function(action){
var item = this.toolbar.getItem(action);
if (item) {
var dblclick = MooEditable.Actions[action]['ondblclick'];
if (typeOf(dblclick) == 'function'){
dblclick.attempt([item,e], this);
return;
}
}
}.bind(this));
this.fireEvent('editorDoubleClick', [e, this]);
},

This would allow custom buttons to have a double click feature. For example, the createlink button can be extended so that if the users double clicks on a link then the link dialog will open up for edting.

@cheeaun
Copy link
Owner

cheeaun commented Oct 31, 2010

Fixed in bb58ac8

Not closing this issue since the double click feature seems like a good idea. Will be looking into this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants