Closed (fixed)
Project:
Editable Fields
Version:
7.x-1.x-dev
Component:
User interface
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
10 Aug 2011 at 17:54 UTC
Updated:
6 Jun 2015 at 17:21 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
jg314 commentedI realized that this was due to the code in editablefields.js. I have included a patch that will also make checkboxes work without the save button.
All I did was add input[type='checkbox'] to two jQuery selectors.
Comment #2
krlucas commentedMarking as duplicate of #1249730: Hide submit button for radio buttons too. Both patches look nearly identical. Auto submitting radio buttons make a little more sense to me than checkboxes given potential latency issues when I want to check more than one checkbox. Thanks!
Comment #3
mikeytown2 commentedPatch in #1 fixed an issue we were having. Re-rolled for latest dev.
Comment #4
bschilt commentedI have applied the patch in #3 to 7.x-1.0-alpha2+7-dev and it works great!
Comment #5
dags commentedCommitted the patch in #3 to 7.x-1.x. This fixes the issue for views but the regular node display is still off. I imagine we'll want to fix that too.
Comment #7
_vid commentedFor anyone still using 7.x-1.0-alpha2 here's the patch in #3 rerolled for alpha2.
Comment #8
jason.fisher commentedPatch does not work for checkboxes > 1.
Should become:
Comment #9
_vid commented@jason.fisher; I totally agree. Though we went about it in a different way;
I updated that line to allow 1 or more text, checkbox, textarea and selects and more than 1 radio:
if ($this.find('input[type=text],input[type=checkbox],textarea,select').length >= 1 || $this.find('input[type=radio] ').length > 1) {In doing so, we noticed that the 'add more' button would disappear. So we also updated that hide() statement:
$this.find('input.form-submit:not(".field-add-more-submit")').hide();Here's the diff between my patched version in #7 and this:
It may go beyond the scope of this (resolved) issue so I didn't post a fresh patch but let me know what you think of this solution. If it takes care of your multiple checkbox needs then I'll upload a fresh patch for alpha2.
Comment #10
_vid commentedWe're still using the patch #7 and I just re-rolled it from within the editablefields directory.
Essentially the only changes are the path:
a/editablefields/editablefields.module b/editablefields/editablefields.module
Becomes
a/editablefields.module b/editablefields.module
Comment #11
HelloStephanie commentedThe last patch is working for me except not in the Bootstrap theme https://drupal.org/project/bootstrap 7.x-3.0.
Any ideas why it doesn't work with Bootstrap?
Comment #12
HelloStephanie commentedFigured it out. This patch should also include alternate forms of input buttons such as Bootstrap's: "button.form-submit"
Comment #13
vadim_k commenteddoes anyone have any idea how to patch editablefields for autosaving selectboxes created by chosen?
Comment #14
jgullstr commentedIssue related to bootstrap: #1909768: Editable field is hidden in Views table themed by Bootstrap theme
Comment #15
cockers commented#10 worked for me, will this be committed?