This is field support for users. A number of things here could be reused for nodes and comments. It breaks AJAX replace even for non-fields. I don't know why. I hope I can find someone to fix it.

CommentFileSizeAuthor
#5 fasttoggle-1214232-5.patch8.04 KBtraviscarden
#2 fasttoggle_fields.patch8.25 KBchx
#1 fasttoggle_fields.patch8.16 KBchx
fasttogle_fields.patch7.96 KBchx

Comments

chx’s picture

chx commented
StatusFileSize
new fasttoggle_fields.patch8.16 KB

Much better, this one. The AJAX support was broken due to calling non-exisitng function base_url().

chx’s picture

chx commented
StatusFileSize
new fasttoggle_fields.patch8.25 KB

*ahem* but colons need escapin' in jQuery selectors.

paulgemini’s picture

paulgemini commented

Thanks for doing this. Forgive my ignorance - could you explain what this feature does?

chx’s picture

chx commented

If you define a list field with two values , you can toggle between them.

traviscarden’s picture

traviscarden commented
StatusFileSize
new fasttoggle-1214232-5.patch8.04 KB

The patch in #2 no longer cleanly applies. Here's a new git patch. I share it in the hope of accelerating the issue, but I didn't actually get it to work for myself (or take the time to). Do you have to implement hook_fasttoggle_labels() for it to find your fields?

nigelcunningham’s picture

nigelcunningham commented

Would someone provide instructions for testing the patch, please?

nigelcunningham’s picture

nigelcunningham commented
Status: Needs review » Fixed

Code just committed to the 7.x-1.x branch adds Fields support.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

fadgadget’s picture

fadgadget commented

Hello i have a simple on/off checkbox text field added to a content type. Is it possible to add this to Fast Toggle now?

thanks

fadgadget’s picture

fadgadget commented
Issue summary: View changes

Added a note that non-fields are broken too