Patch (to be ported)
Project:
jEditable inline content editing
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
23 Mar 2011 at 14:56 UTC
Updated:
13 Feb 2012 at 21:48 UTC
It appears that upon installing this module, another module, Admin Tools, part of my installation become inoperable. It is (Admin Tools) permanently collapsed and I am unable to expand or make any selections. Since that is more important for my day to day functions, I had to disabled this module... Bummer.... Kinda had high hopes for this...
Comments
Comment #1
realOFF commentedI must add that upon enabling it, it disables all collapsible-collapsed funcionalities plus some other things as Views tabbing.
Comment #2
deadcyclo commentedSame problem here. I think this is because drupal_edit.js fails with error (uncought type error) on pages that don't contain editable fields. The module should only include the javascript on pages that do contain editable fields. (Alternatively, though not as good a solution: fix the script).
Comment #3
deadcyclo commentedI've fixed this now. For some reason I cannot clone the git-repo, so I'll just paste what I've done here:
Simply replace the contents of drupal_jeditable.js with:
Drupal.behaviors.jeditable = function(context) { if ($('.jeditable-textfield').length) { $('.jeditable-textfield').editable('/jeditable/ajax/save', { indicator : 'Saving...', tooltip : 'Click to edit...', cancel : 'Cancel', submit : 'Save', style : 'display: inline; min-width: 100px;' }); } if ($('.jeditable-textarea').length) { $('.jeditable-textarea').editable('/jeditable/ajax/save', { type : 'textarea', cancel : 'Cancel', submit : 'OK', indicator : 'Saving...', tooltip : 'Click to edit...' }); } if ($('.jeditable-select').length) { $('.jeditable-select').editable('/jeditable/ajax/save', { loadurl : '/jeditable/ajax/load', type : 'select', submit : 'OK', style : 'display: inline' }); } }Seems that this module has never been maintained (aka, the creator uploaded it and ignored it afterwards). I'm seriously considering cloning this module and extending it. It has potential, but needs a fair amount of work. This bug has existed for almost a half a year now, and it was trivial to fix.
Comment #4
broncomania commentedThx man for sharing. It works on the first view for me.
Comment #5
Rosamunda commentedComment #6
kevin.mcnamee@mailbox.org commentedFixed in 6.x-1.x branch.