Closed (outdated)
Project:
Editable Fields
Version:
6.x-3.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
1 May 2011 at 19:32 UTC
Updated:
31 Jan 2017 at 18:10 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
andreiashu commentedComment #2
andreiashu commentedFirst patch attached.
It supports adding and removing (not just replacing values) on top of what we had initially.
I tried to continue on the direction that the sandbox code took: replace individual values in the UI. This seems to be pretty fragile because we can't render 1 value of a multivalued field along with it's wrapper divs (please correct me if I'm wrong!).
In EF we don't have this issue as we just wrap the whole field in a known div (think #id) and then render and replace the whole thing. This has it's drawbacks too (afaik something to do with the field's label) but it seems to me better than trying to handle the values individually.
Thoughts are welcomed.
Comment #3
andreiashu commentedI forgot to mention: I didn't yet include the views hooks. I'll do that in the future, once we figure out which way to go with the normal node view.
Comment #4
krlucas commentedI just did a brain dump here of my ideas regarding CTools integration at #1040592: Edit in modal frame/pop-up via jQuery ?.
but we should probably continue the conversation here.
One caveat to my thoughts there is we'll also need to write plug-ins for any other fields currently supported by 6.x-2.0
Comment #5
krlucas commentedHere's another version of my brain dump!
So possibly two new plug-in types--editable and responder. The responder is about reducing the duplicate php code required to render and repsond to form rendered in a modal, an inline or some custom AJAX/JS responder using the ctools form and ajax command apis--anticipating that modal and inline will be able to handle many forms provided by most "editable" plug-ins. That said, to reduce the complexity of THIS module, we could treat "responder" as a pattern within an editable plug-in. Each editable plug-in would be larger, doing a lot of the same ctools_includes and and commands and stuff.
Comment #6
joelpittetLast update was 2011, going to close as outdated because it's against 6.x branch.