Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
In admin/content/types -> Locative information -> Collection settings, the drag and drop of fields have no effect.
Comment | File | Size | Author |
---|---|---|---|
#16 | tabledrag_js_16.patch | 1.96 KB | q0rban |
#10 | jquery_update_tabledrag.patch | 39.4 KB | q0rban |
#9 | jquery_update_tabledrag.patch | 39.38 KB | q0rban |
#7 | tabledrag.js_.patch | 937 bytes | bdragon |
#1 | Picture 7.png | 81.86 KB | drewish |
Comments
Comment #1
drewish CreditAttribution: drewish commentedYeah and the weights are showing up. It looks like somethings kind of screwy. see the attached image.
Comment #2
bdragon CreditAttribution: bdragon commentedDrag and drop was fixed just before the 3.0 release. (Two seperate bugs were preventing it from working properly.)
re: drewish's issue: I'm guessing a JS failure somewhere? I can't reproduce..
Comment #3
drewish CreditAttribution: drewish commentedSeen that on two sites and no JS errors according to firebug.
Comment #4
bdragon CreditAttribution: bdragon commentedWhat release of Drupal 6? Perhaps something changed with the weight fields somewhere along the line... The javascript is supposed to hide the weight field....
Comment #5
bdragon CreditAttribution: bdragon commentedReopening because I was able to reproduce on a test site running a newer Drupal. I had been testing against 6.6.
Comment #6
bdragon CreditAttribution: bdragon commentedDOH! I had a local mod to work around a core bug, and totally forgot about it!
Comment #7
bdragon CreditAttribution: bdragon commentedHere's the core patch I've been running on.
The problem is that tabledrag.js malfunctions if the tabledrag is inside a collapsible fieldset, because it checks whether an element is hidden already before hiding it, and the way it checks considers it "hidden" if it's just inside a collapsed fieldset.
This should get fixed in core I guess.
Comment #8
q0rban CreditAttribution: q0rban commentedI see this also on multiple filefields inside of a collapsed fieldset. Perhaps this could be fixed with jquery update by changing this to a feature request for that project.
Comment #9
q0rban CreditAttribution: q0rban commentedMoving this over to jquery_update and attaching a patch. jQuery Update will only override the file if you have the settings to something other than 'Packed' or are using a jQuery version greater than the one that ships with d6.x.
To describe the problem for the jquery_update maintainers, if you have tabledrag elements inside of a collapsed fieldset, the tabledrag js breaks and doesn't hide the weight fields or add the handles.
Comment #10
q0rban CreditAttribution: q0rban commentedWhoops, just noticed I put the tabledrag.js in the wrong folder.
Comment #11
mfer CreditAttribution: mfer commentedI added this to the drupal 6 jQuery Update 1.x dev branch. I would really like this to be tested some more before I release it. I did some quick tests in IE6/7, FF, Safari, and Chrome and didn't find any problems. Is there a reason it was not applied to hidden areas in the past?
This is really an issue that should be fixed in drupal core so I am moving it there.
Comment #12
mfer CreditAttribution: mfer commentedadding javascript tag
Comment #13
mfer CreditAttribution: mfer commentedNow that I look at this, why does tabledrag work for the upload module in a fieldset and not Location?
Comment #14
mfer CreditAttribution: mfer commentedSpoke with q0rban and in the case of the upload module it works because that fieldset is always open when there are files in there or being added. If it is forced to be closed the tabledrag is never attached to these.
I'm not sure the implementation suggested in these patches is a good idea. There is potential, though unlikely, for it to explode on some hidden content in a page. I wonder if attaching the tabledrag behavior to a fieldset when it is initially opened would be a better idea.
Comment #15
q0rban CreditAttribution: q0rban commentedSwitching back to a bug report and updating the title. The current patch breaks on menu administration (ie, admin/build/menu-customize/navigation), as it removes the 'Enabled' column header.
Comment #16
q0rban CreditAttribution: q0rban commentedSafari now honors display:none on th, so switching to that fixed the above issue.
Comment #18
Kars-T CreditAttribution: Kars-T commentedPlease change these lines to
and auto testing should succeed. Otherwise HEAD did change a lot and I can't apply and test your patch manually.
This review is powered by Dreditor.
Comment #19
udig CreditAttribution: udig commented#425148: Form elements within a collapsed field group are not visible in Internet Explorer was marked as duplicate for this issue.
Is it?
The symptoms mentioned here seems to be different then these mentioned over there.
?
Comment #20
q0rban CreditAttribution: q0rban commentedMarking this as duplicate of #303189: Tabledrag doesn't hide columns when the whole table is initially hidden